search

nfdi4plants / ARCCommander

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

[Bug] `arc sync` #147

Closed bvenn closed 11 months ago

bvenn commented 2 years ago

When working on a feature branch arc sync should target that branch rather than trying to push to the main branch. Of course you can specify the branch using arc sync -b <branchName> but if you already are on that branch, it should be chosen.

Additional context Current workflow:

//changes on branch "myBranch"
//git status validates beeing on "myBranch"
arc sync -m "add xy"
//changes are pushed to main branch
//git status validates beeing on "main"
omaus commented 2 years ago

Good idea. But maybe this is also up to debate since this is, imo, a matter of habit, too. But in general, I like your idea and would also prefer it to be that way (maybe with a warning that this is not the main branch, to tackle cases where the user forgot that they switched the branch?). @HLWeil what do you think?

HLWeil commented 2 years ago

This is actually how it's supposed to work. So I'll mark this issue as a bug instead. Thanks for the heads up, @bvenn