I think it's fine. I was debating between roof:shape=many and roof:shape=gabled. I went for gabled in the end, but can be split into building parts later to be more accurate. I think I just messed up setting the tags in JOSM!