nfdi4plants / ARCCommander

Tool to manage your ARCs
MIT License
11 stars 9 forks source link

[Feature Request]Rename repo/API or tool #208

Closed kMutagene closed 9 months ago

kMutagene commented 12 months ago

There are imo too many different naming conventions across our repos. However, some patterns have emerged, which i quite like:

so my suggestion would be renaming the API from ArcCommander to ARCCommander, and the tool from arc to arc-commander.

If these changes are too big, I think we should at least rename the repo accordingly, since the current name - arcCommander - does not follow any pattern mentioned above, but is also differing from the current tool name (arc) and API namespace (ArcCommander).

kMutagene commented 12 months ago

See also https://github.com/nfdi4plants/arc-validate/issues/42

HLWeil commented 12 months ago

I agree with renaming the repo to ARCCommander. But I think we should stick with the tool name arc for ease of use.

kMutagene commented 12 months ago

@HLWeil do you intend to publish the API used by the tool as nuget package? If so, i think you should also rename the API namespace. Otherwise it does not really matter.

HLWeil commented 12 months ago

Can you specify what you mean with API?

kMutagene commented 12 months ago

https://github.com/nfdi4plants/arcCommander/tree/developer/src/ArcCommander/APIs

kMutagene commented 12 months ago

namespace ArcCommander.APIs

kMutagene commented 12 months ago

or is that basically obsolete now that ARCtrl exists?

omaus commented 12 months ago

or is that basically obsolete now that ARCtrl exists?

That was the plan (at least to my knowledge).

HLWeil commented 12 months ago

Ah I see. Yeah this definitely won't make it into its own nuget package. It will just be a thin layer combining ARCtrl top-level functions.

HLWeil commented 9 months ago

Renamed repository to ARCCommander