Skip to content
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

Adding keywords in OPAM files #1123

Merged
merged 1 commit into from
Jan 10, 2024
Merged

Adding keywords in OPAM files #1123

merged 1 commit into from
Jan 10, 2024

Conversation

proux01
Copy link
Contributor

@proux01 proux01 commented Nov 8, 2023

Motivation for this change

Add keywords in OPAM files.

Following last meeting https://github.com/math-comp/math-comp/wiki/Minutes-November-08-2023#adding-more-keywords-in-opam-file

Cc @CohenCyril @Tragicus @affeldt-aist @ybertot @pi8027 @gares who attended the meeting

Things done/to do
  • added corresponding entries in CHANGELOG_UNRELEASED.md
  • added corresponding documentation in the headers
  • tried to abide by the contribution guide
Compatibility with MathComp 1.X
  • I added the label TODO: MC-1 port to make sure someone ports this PR to
    the mathcomp-1 branch or I already opened an issue or PR (please cross reference).
Automatic note to reviewers

Read this Checklist and put a milestone if possible.

@proux01 proux01 added this to the 2.2.0 milestone Nov 8, 2023
@proux01
Copy link
Contributor Author

proux01 commented Nov 8, 2023

I only added keywords for ssreflect and algebra as I feel a lot less qualified about fingroup, solvable, field and character.
Feel free to extend / change keywords (either by pushing commits or commenting below).

@proux01
Copy link
Contributor Author

proux01 commented Nov 22, 2023

@amahboubi your name was mentionned during the last meeting https://github.com/math-comp/math-comp/wiki/Minutes-November-22-2023 as our expert on character. If you want to add some keywords for that part, it would be great. Otherwise no worry this could still be merged as is.

@proux01
Copy link
Contributor Author

proux01 commented Jan 10, 2024

Nothing happened since end of November, let's merge as it is. We can still add more later.

@proux01 proux01 merged commit 36ec398 into master Jan 10, 2024
3 of 144 checks passed
@proux01 proux01 deleted the opam_keywords branch January 10, 2024 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants