-
Notifications
You must be signed in to change notification settings - Fork 23
Update README.maintainer.md for GAP updates
#1281
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
base: master
Are you sure you want to change the base?
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #1281 +/- ##
=======================================
Coverage 76.55% 76.56%
=======================================
Files 61 61
Lines 4901 4902 +1
=======================================
+ Hits 3752 3753 +1
Misses 1149 1149 🚀 New features to boost your workflow:
|
| 7. In `GAP.jl`, create a PR with the following changes: | ||
| 1. In the `Project.toml` update the compat bounds as follows: | ||
| - `GAP_jll` to the `version` from step 1, with a `~` prefix. | ||
| - `GAP_lib_jll` to the `version` from step 3, with a `~` prefix. | ||
| - %TODO: `GAP_pkg_juliainterface` | ||
| - For each `GAP_pkg_*_jll` package that was updated in step 6, update to the new version | ||
| with a `~` prefix. You can find the new version numbers in the registry PRs from step 6. | ||
| 2. Run `etc/update_artifact.jl` with the new GAP version as argument to the `Artifacts.toml` file. | ||
| See the top of that script for instructions. | ||
| 3. If the GAP release is not ABI-compatible with the previous one, update the minor part of the version | ||
| of `GAP.jl` in its `Project.toml` | ||
| 4. If the GAP release was created from a new release branch (e.g. `stable-4.15` instead of `stable-4.14`), | ||
| then update occurrences in `.github/workflows/gap.yml` accordingly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perhaps we can write a script to take care of most of this, given just the new GAP version? (Not for this PR! just thinking out loud)
Follow-up to #1203 with the learnings that I made during the GAP 4.15.0 and 4.15.1 updates.
There are two places where I should add another sentence about how
juliainterfaceis special at that place. I'll do that before we merge this, but I wanted to already open this PR to get some feedback.