Skip to content

Minor updates for release documentation #20631

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.

Sign up for GitHub

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

Jump to bottom
Merged
jkeenan merged 1 commit into from
Aug 16, 2023
Merged

Minor updates for release documentation #20631

jkeenan merged 1 commit into from
Aug 16, 2023

Conversation

@richardleach
Copy link
Contributor

@richardleach richardleach commented Dec 20, 2022

These are mostly additions for clarity or extra examples.

Some advice has been removed from "update INSTALL", because the wording within INSTALL has been made less version-specific, requiring fewer updates per-release.

@jkeenan
Copy link
Contributor

jkeenan commented Dec 30, 2022

These are mostly additions for clarity or extra examples.

Some advice has been removed from "update INSTALL", because the wording within INSTALL has been made less version-specific, requiring fewer updates per-release.

Can we get some feedback on these suggestions from other recent monthly dev release managers?

Corion
Corion reviewed Dec 31, 2022
View reviewed changes
@demerphq
Copy link
Collaborator

demerphq commented Feb 8, 2023

@Corion and @karenetheridge, please note that @richardleach is waiting on your feedback on this PR.

@richardleach note that the PR is not passing CI. did you want to modernize it?

@richardleach
Minor updates for release documentation
4d5fb87 
These are mostly additions for clarity or extra examples.

Some advice has been removed from "update INSTALL", because the wording
within INSTALL has been made less version-specific, requiring fewer
updates per-release.
@jkeenan
Copy link
Contributor

jkeenan commented Aug 1, 2023

@Corion and @karenetheridge, please note that @richardleach is waiting on your feedback on this PR.

@richardleach note that the PR is not passing CI. did you want to modernize it?

@Corion, @karenetheridge, @richardleach , can you respond to the questions above? (If we get no response I think the ticket should be closed.)

@jkeenan jkeenan added Closable? We might be able to close this ticket, but we need to check with the reporter Infrastructure Things needed to maintain Perl development labels Aug 1, 2023
@Corion
Copy link

Corion commented Aug 1, 2023

The changes look good to me now

@jkeenan jkeenan removed the Closable? We might be able to close this ticket, but we need to check with the reporter label Aug 16, 2023
@jkeenan jkeenan merged commit cf56d3c into Perl:blead Aug 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Corion Corion Corion left review comments

@toddr toddr Awaiting requested review from toddr

@karenetheridge karenetheridge Awaiting requested review from karenetheridge

Assignees

No one assigned

Labels

Infrastructure Things needed to maintain Perl development

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@richardleach @jkeenan @demerphq @Corion