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

Updated release checklist #3385

Merged
merged 11 commits into from
Aug 27, 2024

Conversation

mahrud
Copy link
Member

@mahrud mahrud commented Jul 31, 2024

  • limited package headlines to 100 characters
    • also fixes the top node check, which wasn't working.
  • removed Reload => true from packages
  • implemented uniformity in headlines (cf. 7a7e6f9)
  • removed line breaks from package headlines

I'll seek approval from authors of affected packages in the comments.

@mahrud mahrud marked this pull request as ready for review August 23, 2024 20:03
@mahrud mahrud requested a review from d-torrance August 23, 2024 20:03
@d-torrance
Copy link
Member

I just pushed a commit that should fix the RInterface error (we were missing the top level node when R is missing during the build).

@d-torrance
Copy link
Member

Is this ready to merge?

@mahrud
Copy link
Member Author

mahrud commented Aug 27, 2024

I think everyone I tagged responded.

@mahrud mahrud merged commit 5db7db6 into Macaulay2:development Aug 27, 2024
5 checks passed
@mahrud mahrud deleted the quickfix/release-checklist branch August 27, 2024 15:55
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.

5 participants