-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/configuration file #5
base: main
Are you sure you want to change the base?
Conversation
maybe it would be better to preserve history? Rather than collapse it in a colouring-cities/colouring-core@2e784ce commit? Basically, I would setup this repository as copy of colouring London repository as this allows to avoid loss of history, commits and so on. I look often at commits made to files as it allows to understand why some changes were made and who can be consulted about design decision that were made, look at related issues and so on. |
Maybe it would be a good idea to migrate colouring-london repository to colouring-cities repository, migrate London specific content into config file? |
Beyond the first commit, changes in this PR are all docs and fixing some links/references, look good 👍 Agreed with @matkoniecz that we should preserve history. Two options:
I think I prefer 2 - keeps the record of code next to history of issues and PRs; is less of a fresh start but means the focus moves clearly to the “core”. Assumes we’re confident in the configuration code and avoids working on a(nother) fork for an extended period. |
Yes, preserving issue/PR numbers is quite helpful so (2) also seems preferable to me. It has some negatives, but as project gets larger ability to look at file history/past commits/ |
Yeah, to be honest, I've been struggling to get my head around this. Ultimately, we want this to be the base repository and for Colouring London (and all other "Colouring..." projects) to spin off from it. So are we saying it's better to delete this repo, then rename "Colouring London" to "Colouring Core", and then create a new "Colouring London" repo as a Fork from it? I'm assuming Forking is the best way to do it? I haven't worked on a project with this many different versions/repos before. How easy is it to fold changes from the forked "Colouring London" back into the Core repo? How will this affect all of the other projects that are currently forked from Colouring London? I just want to make sure I do it right and don't mess it up for everyone before I press "go"! |
Yes, though I would also migrate issues and wiki (and maybe something else). Maybe rename to colouring-cities/archive-colouring-core-first-attempt and archive rather than deleting to keep for example this discussion.
Rename and creation a new repository under old name may result in some links, including remotes in git repositories to end in weird state. But as new repo will be fork of previous one and still contain the same commits it should not be a problem, right?
This can be done with multiple remotes and merging/cherry picking. It is easy in CLI git and should be easy in decent git GUIs. Though I expect that changes usable in Colouring Core would be made against Core repository, and Colouring London would merge upstream. A bit different situation, but I do something similar in https://github.com/matkoniecz/Zazolc/commits/fork There is https://github.com/streetcomplete/StreetComplete which is the main repository, and https://github.com/matkoniecz/Zazolc which is a fork that continues to merge in upstream. In my local repository I added https://github.com/matkoniecz/Zazolc as I have branch I have branch I contribute to https://github.com/streetcomplete/StreetComplete changes benefiting entire project. Fork gets commits with my own changes which for various reasons are ineligible for main version. So scenario with work on fork that uses all options would have something like
this will commit some local changes, merge local brach and merge in new commits upstream Typical one would be just regular If I would be adding something upstream - committing would be like normal. |
Yes, I think this is the thing to do. (Edit: as someone said above, actually rename this repo instead of deleting) I think the situation described with multiple remotes, merging from upstream, is a reasonable solution only if we continue to need different code for each platform, and it might be a good first next step.. ..but we should be aiming for a “Colouring core application” that can be deployed and configured, so for example the only reason to keep a “Colouring London” repository is to keep the configuration in version control - deployment is then something like:
|
Yes, that would be even better - though not immediately available. |
hello ;-) Question: I was also wondering, what could be the best way of updating e.g. Colouring Dresden repo from Colouring Core ? so, after important changes in Colouring Core, I can start the syncing manually? do you have also other suggestions? |
Hi @traveller195. Yes, I think that's the best way to do it, and I believe it's how the other projects are set up, but I haven't worked a lot with forked projects before. |
I'm going to begin the process of renaming everything now, starting with this repo. I'll just leave this pull request as it is for now. |
WIP - Currently Untested
The latest version of the code from colouring-london/colouring-core, with additional documentation on the config files.