Decide on File naming for RxCocoaExt(s) and RxSwiftExt(s)

[freak4pc]

Description

This ticket continues discussion on one of the point form #127.

Would appreciate some opinions on what you guys would consider proper naming for files.

In an optimal world we could keep the same naming across RxSwift and RxCocoa components.

e.g.

• RxSwift
  • distinct.swift
• RxCocoa
  • distinct.swift

But swiftc won't compile two filenames of the same name, even if they don't contain any definition conflicts.

Right now the naming seems to follow

• RxSwift
  • distinct.swift
• RxCocoa
  • distinct+RxCocoa.swift

Does anyone have any nicer ideas of how to name these files ?
Thoughts I sort of have ATM:

• distinctSwift.swift, distinctCocoa.swift might be a bit meh?
• having a single distinct.swift file in a Common folder (same level as RxSwift and RxCocoa) for every filename that has extensions that are relevant for both. But actually this won't work since there won't be a good way to subspecies these... (maybe build flags)

This is a bit of a tricky subject so would appreciate any help.

Anyways, For tests It would just be the same with Tests.swift appended to the end of whatever filename, to keep things consistent.

Also for naming in general, seems there was an agreement for:

• Class Case for classes (including XCTestCase subclasses)
• camelCase for operators

Thanks for any assistance in naming :)

[fpillet]

IMHO, the distinct+RxCocoa.swift form is the tried and true way of naming category files in Obj-C and has worked well for years. We should use it wherever the file name overlaps an existing one from the Swift folder (considered the baseline for file names).

Regarding tests, the Tests.swift extension look good to me as well.

Casing rules are 👍

[freak4pc]

Thanks for taking the time to comment @fpillet :)

the distinct+RxCocoa.swift form is the tried and true way of naming category files in Obj-C

In general I agree. The problem is we aren't naming the RxSwift counterparts as distinct+RxSwift - meaning RxSwift is "main" and RxCocoa is secondary in some sort? Not sure if that's a problem but wanted to bring it out there.

[fpillet]

This is actually my idea. You can't have RxCocoa without RxSwift anyway, so it doesn't make real sense (other than purely aesthetic) to force naming distinct+RxSwift.swift.

Another idea I have, since RxCocoa is all about UI, would be to name distinct+UI.swift. What do you think?

[freak4pc]

Actually that's a fair point about RxSwift being the "base".

I think since distinct isn't applied specifically to UI components, but just to these RxCocoa additions, we can keep the current +RxCocoa.swift naming. Sounds good?

[fpillet]

Sure, sounds good!

[mosamer]

@freak4pc Sorry for late reply, but I think it is possible to have same file names on RxSwiftExt/RxCocoaExt (which is optimal IMO) once they are separated into two projects (as per the existing plan). swift error is happening when the 2 files belong to the same target. Here is a link to a sample workspace I believe we should finally look similar to.