-
Notifications
You must be signed in to change notification settings - Fork 74
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
generate software/compiler compatibility table automatically #2407
Comments
@psychocoderHPC What do you think about a bot, which generates the table and push it to the repository before a PR is merged? So it means the bot automatically generate/update the markdown file with the table which we simply link in the README. |
Do you mean to generate the table based on what the CI tests, or based on which tests actually pass ? |
I like the idea. I am not sure if I would invest time into a bot. I would be happy to have a script like the current generator which uses the matrix job generator instead of the hand written json. |
yes
I agree that it should be not bot, it should be a GitHub Actions. Something which needs no extra system to maintain. I thought about, that the table is generated when we hit the merge button. Than it simply generates the table, do a commit, maybe squash and afterwards the actual merge is done. Therefore the change table will not appears in PR but on the dev branch. |
I did a short check. Looks like I can do a CI job during the merge process: https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/configuring-pull-request-merges/managing-a-merge-queue#triggering-merge-group-checks-with-github-actions
|
Keep the software/compiler compatibility table up to date is hard to do. I already implemented a script, to generate the markdown table from a json config file.
The job generator already know all valid combinations, except for MacOS and Windows. Therefore it makes sense to combine the table generator script with the job generator to automatically generates the compatibility table for Linux.
The text was updated successfully, but these errors were encountered: