Perfect 👍

Edit: actually Paths.swift will conflict with a --merged generation since the Paths file is Paths.swift

Edit: actually Paths.swift will conflict with a --merged generation since the Paths file is Paths.swift

Oof good point! Now I'm wondering why the tests didn't flag that 😄

Edit: Ah, it overwrote it 😄

I wonder if its easiest here to just go back on what we said earlier and continue to embed the extensions inside the merged file? I'm trying to remember why I suggested that and I think the only real reason was because I thought it would be easier and would push people towards not merging sources.

Any concerns with that? Alternatively I could think of a better name, but I've got nothing.

I think that's good for merged. However, I think there's a good probability that a few schemas exist that have a Paths object which would conflict. We can ignore that for now and wait until somebody raises an issue.

Yeah at a minimum the generator should probably spot that conflict and error, but I'm looking at the file output formatting code now and again I think we could doing some refactoring in this area to clean things up nicely.

I'll merge them in for the time being but will keep this in mind 👍

Actually, an object called Paths would be written to ./Paths/Paths.swift when not merging, but the Paths namespace would be written to ./Paths.swift.. So while the files wouldn't overwrite each other, the two types would conflict and the user would have to either change the namespace or rename the entity 😄

Actually, what if we changed the namespace to be the name of the package/module by default? This would only affect paths but that's a correct expectation. The directory Paths/ would stay the same as it holds paths.

import PetStoreAPI

let petsRequest = PetStoreAPI.getPets()

Edit: scratch that, there's a bug in the compiler when using the same name as the module for namespacing purposes: swiftlang/swift#43510. I remembered OpenAPITools works around this issue by making names somewhat dumb. Wouldn't hurt to make sure though.

Edit: Ah, it overwrote it 😄

Wasn't possible with "+" in file names 😄

Reverted the behaviour when using --merge-sources so that it worked like it did before #95 (extensions are now included in the merged sources again).

They're only placed outside of the Entities/Paths group when generating them in split mode. It avoids naming collisions and lets us stick with more natural names