Skip to content

Deprecate non boolean if syntax - #22123

Merged
coqbot-app[bot] merged 1 commit into
rocq-prover:masterfrom
proux01:deprecate-if
Jul 3, 2026
Merged

Deprecate non boolean if syntax#22123
coqbot-app[bot] merged 1 commit into
rocq-prover:masterfrom
proux01:deprecate-if

Conversation

@proux01

@proux01 proux01 commented Jun 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Deprecate the non boolean if t then _ else _ syntax (in favour of if t is <first_constructor> then _ else _).

Depends on: #21609, #22124 (merged)

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

@proux01 proux01 added needs: merge of dependency This PR depends on another PR being merged first. kind: deprecation Deprecation labels Jun 12, 2026
@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 12, 2026
@proux01 proux01 mentioned this pull request Jun 12, 2026
Merged
14 tasks
@proux01 proux01 added the request: full CI Use this label when you want your next push to trigger a full CI. label Jun 12, 2026
@coqbot-app coqbot-app Bot removed request: full CI Use this label when you want your next push to trigger a full CI. needs: full CI The latest GitLab pipeline that ran was a light CI. Say "@coqbot run full ci" to get a full CI. labels Jun 12, 2026
@proux01 proux01 added this to the 9.3+rc1 milestone Jun 12, 2026
@proux01 proux01 added the request: full CI Use this label when you want your next push to trigger a full CI. label Jun 12, 2026
@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 Jun 12, 2026
@SkySkimmer

SkySkimmer commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Is printing of non booleans changed to use the new syntax too?

@proux01

proux01 commented Jun 12, 2026

Copy link
Copy Markdown
Contributor Author

I'm working on it (in a separate PR)

@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 12, 2026
@proux01 proux01 added the request: full CI Use this label when you want your next push to trigger a full CI. label Jun 15, 2026
@github-actions github-actions Bot added needs: rebase Should be rebased on the latest master to solve conflicts or have a newer CI run. and removed needs: rebase Should be rebased on the latest master to solve conflicts or have a newer CI run. labels Jun 17, 2026
@proux01
proux01 marked this pull request as ready for review June 30, 2026 06:54
@proux01
proux01 requested review from a team as code owners June 30, 2026 06:54
@github-actions github-actions Bot added the needs: rebase Should be rebased on the latest master to solve conflicts or have a newer CI run. label Jul 1, 2026
@coqbot-app coqbot-app Bot removed request: full CI Use this label when you want your next push to trigger a full CI. needs: full CI The latest GitLab pipeline that ran was a light CI. Say "@coqbot run full ci" to get a full CI. labels Jul 1, 2026
@coqbot-app coqbot-app Bot removed the needs: rebase Should be rebased on the latest master to solve conflicts or have a newer CI run. label Jul 1, 2026
@proux01 proux01 removed the needs: merge of dependency This PR depends on another PR being merged first. label Jul 1, 2026
gares
gares reviewed Jul 1, 2026
View reviewed changes
Comment thread test-suite/output/sort_poly_elab.v
@proux01 proux01 added the request: full CI Use this label when you want your next push to trigger a full CI. label Jul 1, 2026
@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 Jul 1, 2026
@gares

gares commented Jul 3, 2026

Copy link
Copy Markdown
Member

@coqbot merge now

@coqbot-app

coqbot-app Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

@gares: You cannot merge this PR because:

  • You are not among the assignees.

@gares gares self-assigned this Jul 3, 2026
@gares

gares commented Jul 3, 2026

Copy link
Copy Markdown
Member

@coqbot merge now

@coqbot-app

coqbot-app Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

@gares: You can't merge the PR because it hasn't been approved yet.

@gares

gares commented Jul 3, 2026

Copy link
Copy Markdown
Member

@coqbot merge now

@coqbot-app
coqbot-app Bot merged commit 3e79431 into rocq-prover:master Jul 3, 2026
7 of 9 checks passed
@proux01
proux01 deleted the deprecate-if branch July 6, 2026 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gares gares gares approved these changes

Assignees

@gares gares

Labels

kind: deprecation Deprecation

Projects

None yet

Milestone

9.3+rc1

Development

Successfully merging this pull request may close these issues.

3 participants

@proux01 @SkySkimmer @gares