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

Deprecate NZDomain #18539

Merged
merged 1 commit into from Jan 24, 2024
Merged

Deprecate NZDomain #18539

merged 1 commit into from Jan 24, 2024

Conversation

Villetaneuse
Copy link
Contributor

@Villetaneuse Villetaneuse commented Jan 23, 2024

The file NZDomain, while informative and well written, is of little use to the end user and makes NatInt depend on PeanoNat.

  • Added changelog.
  • Added / updated documentation.

@Villetaneuse Villetaneuse requested review from a team as code owners January 23, 2024 15:28
@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 Jan 23, 2024
Comment on lines 11 to 12
Attributes deprecated(since="8.20",
note="Please open an issue if you think this file is of general interest.")
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Attributes deprecated(since="8.20",
note="Please open an issue if you think this file is of general interest.")
Attributes deprecated(since="8.20").

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do you want the same change in the changelog?

Copy link
Contributor

Choose a reason for hiding this comment

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

Why not, the sentence is not very useful.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done, thank you for your suggestions.

The file NZDomain, while informative and well written, is of little use
to the end user and makes NatInt depend on PeanoNat.
@Villetaneuse Villetaneuse added kind: cleanup Code removal, deprecation, refactorings, etc. part: standard library The standard library stdlib. labels Jan 23, 2024
@Villetaneuse
Copy link
Contributor Author

@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 Jan 23, 2024
@proux01 proux01 self-assigned this Jan 24, 2024
@proux01 proux01 added this to the 8.20+rc1 milestone Jan 24, 2024
@proux01
Copy link
Contributor

proux01 commented Jan 24, 2024

@coqbot merge now

@coqbot-app coqbot-app bot merged commit 27a31a9 into coq:master Jan 24, 2024
5 of 7 checks passed
@Villetaneuse Villetaneuse deleted the deprecate_NZDomain branch January 24, 2024 09:57
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: standard library The standard library stdlib.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants