-
Notifications
You must be signed in to change notification settings - Fork 39
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
strip modules for SkyPy release #389
Conversation
I have checked the documentation and all looks good there. Q: why should this PR not be squashed after merging? |
In our commit history on
Not that I can think of, no. We might want to add a description of the process for adding new modules, and the fact that they are merged into release/master once they are "complete", in a second step. |
Marking this as a draft for now while I test it and other possible solutions locally |
Module branches should be named |
This PR temporarily strips out modules that are not part of the upcoming SkyPy release.
Each module is removed in a separate commit, so this PR should not be squashed. After this PR is merge, we will create individual
module/MODULE_NAME
branches that revert one commit each. These branches will continue development of the modules for the next SkyPy releases.Where necessary, we should be able to simply point currently open pull requests to the new branches once they are created. This might require an update of the PR branch but since stripping and each reversion happens sequentially, that should require minimal intervention.