Skip to content

Conversation

@Ch3LL
Copy link
Contributor

@Ch3LL Ch3LL commented Feb 11, 2020

update release process docs

ping @frogunder can you review here please since you are more involved with this process now?

@Ch3LL Ch3LL requested a review from a team as a code owner February 11, 2020 16:13
@ghost ghost requested a review from xeacott February 11, 2020 16:13
@Ch3LL Ch3LL requested a review from frogunder February 11, 2020 16:16
@max-arnold
Copy link
Contributor

Sorry for jumping in, but I think the following (removed) part is quite important:

Notify salt-users and salt-announce google groups when the feature freeze branch creation will occur so they can try to get their work merged.

Even if there is no separate branch, it is still important to notify contributors:

  1. Before feature freeze, so people can get their PRs in before it is too late
  2. Feature freeze start
  3. Feature freeze end

@Ch3LL
Copy link
Contributor Author

Ch3LL commented Feb 12, 2020

ya thats a fair point. ping @sagetherage just want to make sure that is on the current release process to notify of feature freeze still? obviously we dont need to notify of new branch anymore, just when we plan to not allow any more features in.

@sagetherage
Copy link
Contributor

yes, we will get the communication msg corrected in the auto msg (and will look if there is more). we can keep that verbiage in this doc and update it to be something more like:
"Notify salt-users and salt-announce google groups when the start of the freeze will occur so they can try to get their work merged, and when it will end and merging to continue." Open to suggestions though.

@dwoz dwoz merged commit 291759c into saltstack:master Apr 12, 2020
@sagetherage sagetherage added ZRelease-Sodium retired label documentation Relates to Salt documentation labels May 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Relates to Salt documentation ZRelease-Sodium retired label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants