Skip to content

Deprecate the nonuniform attribute#17716

Merged
coqbot-app[bot] merged 2 commits intorocq-prover:masterfrom
proux01:deprecate_nonuniform
Jun 15, 2023
Merged

Deprecate the nonuniform attribute#17716
coqbot-app[bot] merged 2 commits intorocq-prover:masterfrom
proux01:deprecate_nonuniform

Conversation

@proux01
Copy link
Contributor

@proux01 proux01 commented Jun 9, 2023

Now subsumed byt the #[warning="-uniform-inheritance"] attribute introduced in #16902 .
Following a remark from @Alizter in #17713

  • Added / updated test-suite.
  • Added changelog.
  • Added / updated documentation.
    • Documented any new / changed user messages.
    • Updated documented syntax by running make doc_gram_rsts.
  • Opened overlay pull requests.

Overlays

@proux01 proux01 requested review from a team as code owners June 9, 2023 12:57
@coqbot-app coqbot-app bot added the needs: full CI The latest GitLab pipeline that ran was a light CI. Say "@coqbot run full ci" to get a full CI. label Jun 9, 2023
@proux01 proux01 force-pushed the deprecate_nonuniform branch from d62a7b6 to b0fcc20 Compare June 9, 2023 14:15
@Alizter
Copy link
Contributor

Alizter commented Jun 9, 2023

You could make nonuniform an "alias" of the warning attribute, still deprecate it and also cleanup vernac now. Unless it really is doing something different.

Now subsumed byt the #[warning="-uniform-inheritance"] attribute
introduced in rocq-prover#16902 .
@proux01 proux01 force-pushed the deprecate_nonuniform branch from b0fcc20 to c68837b Compare June 9, 2023 15:28
@proux01
Copy link
Contributor Author

proux01 commented Jun 9, 2023

You could make nonuniform an "alias" of the warning attribute, still deprecate it and also cleanup vernac now. Unless it really is doing something different.

I see, you want me to write the overlays now ;) but you're right, let's do that.

@proux01
Copy link
Contributor Author

proux01 commented Jun 9, 2023

@coqbot run full ci

@coqbot-app coqbot-app bot removed the needs: full CI The latest GitLab pipeline that ran was a light CI. Say "@coqbot run full ci" to get a full CI. label Jun 9, 2023
@jfehrle
Copy link
Member

jfehrle commented Jun 9, 2023

I take it uniform-inheritance is not a new attribute but is only valid within warnings? See #17717

@proux01
Copy link
Contributor Author

proux01 commented Jun 10, 2023

I take it uniform-inheritance is not a new attribute but is only valid within warnings? See #17717

Yes, it's the name of the warning the attribute nonuniform was silencing. That warning has been there for long now.

proux01 added a commit to proux01/coq-elpi that referenced this pull request Jun 10, 2023
@proux01
Copy link
Contributor Author

proux01 commented Jun 10, 2023

CI green

@proux01 proux01 added kind: cleanup Code removal, deprecation, refactorings, etc. part: attributes #[attributes] modify the behaviour of vernac sentences. labels Jun 10, 2023
@proux01 proux01 added this to the 8.18+rc1 milestone Jun 10, 2023
@proux01
Copy link
Contributor Author

proux01 commented Jun 15, 2023

@Alizter would you be the assignee? this is ready

@Alizter Alizter self-assigned this Jun 15, 2023
@Alizter
Copy link
Contributor

Alizter commented Jun 15, 2023

@coqbot merge now

@coqbot-app coqbot-app bot merged commit 6213a21 into rocq-prover:master Jun 15, 2023
@coqbot-app
Copy link
Contributor

coqbot-app bot commented Jun 15, 2023

@Alizter: Please take care of the following overlays:

  • 17716-proux01-deprecate_nonuniform.sh

@proux01 proux01 deleted the deprecate_nonuniform branch June 15, 2023 08:41
bors bot added a commit to Mtac2/Mtac2 that referenced this pull request Jun 15, 2023
388: Adapt to rocq-prover/rocq#17716 r=Janno a=proux01

Adapt to rocq-prover/rocq#17716

To be merged in sync with the upstream PR.

Co-authored-by: Pierre Roux <pierre.roux@onera.fr>
bors bot added a commit to Mtac2/Mtac2 that referenced this pull request Jun 15, 2023
388: Adapt to rocq-prover/rocq#17716 r=proux01 a=proux01

Adapt to rocq-prover/rocq#17716

To be merged in sync with the upstream PR.

Co-authored-by: Pierre Roux <pierre.roux@onera.fr>
bors bot added a commit to Mtac2/Mtac2 that referenced this pull request Jun 15, 2023
388: Adapt to rocq-prover/rocq#17716 r=proux01 a=proux01

Adapt to rocq-prover/rocq#17716

To be merged in sync with the upstream PR.

Co-authored-by: Pierre Roux <pierre.roux@onera.fr>
gares added a commit to LPCIC/coq-elpi that referenced this pull request Jun 15, 2023
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. part: attributes #[attributes] modify the behaviour of vernac sentences.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants