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

Turn Impargs.implicit_status into a record. #18312

Merged
merged 2 commits into from
Nov 16, 2023

Conversation

ppedrot
Copy link
Member

@ppedrot ppedrot commented Nov 14, 2023

Trivial cleanup on my way to sorting out the mess we currently have with evar kinds.

Overlays:

@ppedrot ppedrot added kind: cleanup Code removal, deprecation, refactorings, etc. request: full CI Use this label when you want your next push to trigger a full CI. labels Nov 14, 2023
@ppedrot ppedrot requested review from a team as code owners November 14, 2023 12:51
@coqbot-app coqbot-app bot removed the request: full CI Use this label when you want your next push to trigger a full CI. label Nov 14, 2023
@SkySkimmer SkySkimmer self-assigned this Nov 14, 2023
@SkySkimmer SkySkimmer added this to the 8.19+rc1 milestone Nov 14, 2023
impl_max : maximal_insertion;
impl_force : force_inference;
}

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we use a module to qualify this instead?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not a huge fan of this style, it makes grep unusable.

@SkySkimmer SkySkimmer added the needs: overlay This is breaking external developments we track in CI. label Nov 15, 2023
ppedrot added a commit to ppedrot/coq-elpi that referenced this pull request Nov 15, 2023
ppedrot added a commit to ppedrot/coq-serapi that referenced this pull request Nov 15, 2023
ppedrot added a commit to ppedrot/vscoq that referenced this pull request Nov 15, 2023
@ppedrot ppedrot added the request: full CI Use this label when you want your next push to trigger a full CI. label Nov 15, 2023
@coqbot-app coqbot-app bot removed the request: full CI Use this label when you want your next push to trigger a full CI. label Nov 15, 2023
@ppedrot ppedrot removed the needs: overlay This is breaking external developments we track in CI. label Nov 16, 2023
@SkySkimmer
Copy link
Contributor

@coqbot merge now

@coqbot-app coqbot-app bot merged commit f8e2662 into coq:master Nov 16, 2023
5 checks passed
Copy link
Contributor

coqbot-app bot commented Nov 16, 2023

@SkySkimmer: Please take care of the following overlays:

  • 18312-ppedrot-detuplify-impargs.sh

@ppedrot ppedrot deleted the detuplify-impargs branch November 16, 2023 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind: cleanup Code removal, deprecation, refactorings, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants