-
-
Notifications
You must be signed in to change notification settings - Fork 280
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
docs: add release process instruction #577
Merged
Merged
Changes from 9 commits
Commits
Show all changes
10 commits
Select commit
Hold shift + click to select a range
84c0430
docs: add release process instruction
derberg 7cd44ef
better wording
derberg bce2859
more details on release branch
derberg abfdc83
section about release notes and some grammar fixes
derberg 01e5409
Merge branch 'master' into releaseprocess
derberg 21e2797
finalize release process
derberg 6b4e8a8
codeowners
derberg a509e23
Apply suggestions from code review
derberg cccaefa
feat! instead of !feat
derberg 344e831
Merge branch 'master' into releaseprocess
derberg File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Validating CODEOWNERS rules …
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
# This file provides an overview of code owners in this repository. | ||
|
||
# Each line is a file pattern followed by one or more owners. | ||
# The last matching pattern has the most precedence. | ||
# For more details, read the following article on GitHub: https://help.github.com/articles/about-codeowners/. | ||
|
||
# The default owners | ||
* @github-actions[bot] | ||
|
||
# The default owners are automatically added as reviewers when you open a pull request unless different owners are specified in the file. | ||
/spec/asyncapi.md @fmvilas @derberg |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,105 @@ | ||
# Release Process of the AsyncAPI Specification | ||
|
||
This document is meant to explain the release process of the AsyncAPI specification. This document aims to describe all details of the process so that any member of the community can jump in and help coordinate. | ||
|
||
## Release coordinator | ||
|
||
AsyncAPI specification release involves not only spec release but also the release of AsyncAPI-maintained tools. Therefore it requires a single person called `release coordinator` to ensure the release goes well throughout all phases. | ||
|
||
The release coordinator doesn't have to do all the work alone. The release coordinator needs to find the right people to do the work required for a given phase and engage as many people as needed. | ||
|
||
## Release cadence | ||
|
||
AsyncAPI releases happen regularly in the following months: | ||
- January | ||
- April | ||
- June | ||
- September | ||
|
||
Regular releases concern only major and minor releases. We do not decide up front if the next release is major or minor. This decision depends on the proposals for changes in the specification and how much they affect specification and tooling in a given release cycle. | ||
|
||
Patch releases are automatically released and published. | ||
|
||
## Release phases | ||
|
||
### Kick off | ||
|
||
We start by creating release branches and a placeholder for release notes in the AsyncAPI Blog. | ||
|
||
#### Release branch | ||
|
||
At the beginning of the release cycle, we need to have a new release branch created in some repositories. The reason is that [contribution guide](CONTRIBUTING.md) for the specification requires changes in different projects to accept a proposal. The following repositories are involved: | ||
- [spec](https://github.com/asyncapi/spec) where contributor works with the specification file, | ||
- [spec-json-schemas](https://github.com/asyncapi/spec-json-schemas) where contributor pushes changes to JSON Schema of the spec, | ||
- [parser-js](https://github.com/asyncapi/parser-js) where contributor makes necessary changes in the JavaScript Parser. | ||
|
||
Release branch must have a year and a month of the release as prefix: {YEAR_OF_RELEASE}-{MONTH_OF_RELEASE}-release. For example, a release created in September 2021 has a `2021-09-release` release branch. | ||
|
||
<img src="./assets/release_process/create_branch.png" alt="This image shows part of the GitHub UI that shows how you can create a new branch using default branch as a base." height="300"> | ||
|
||
#### Release notes | ||
|
||
Changes in the specification need to be well described. We need clear information on what has changed, why, and who contributed to the change. A regular changelog is not enough as it is not user-friendly. Every release must have release notes. | ||
|
||
A draft pull request with release notes must be opened at the same time release branches are created. Work on release notes should be done on a feature-by-feature basis and not at the end of the release cycle. Cooperate with contributors. They should be able to provide input and also be allowed to work as release notes article co-authors. | ||
|
||
In other words, once a feature is introduced in the release branch, make sure it is properly described in release notes. | ||
|
||
The draft pull request must be opened against the [Website](https://github.com/asyncapi/website/) repository. Make sure the option `Allow edits and access to secrets by maintainers` is selected to enable support from maintainers. | ||
|
||
<img src="./assets/release_process/draft_pr.png" alt="This image shows example pull request created in GitHub with release notes for AsyncAPI specification" height="300"> | ||
|
||
### Review and merge | ||
|
||
There are no step-by-step instructions to follow but a set of rules. | ||
|
||
#### Review | ||
|
||
- Everybody is invited to perform a review of proposals. The [contribution guide](CONTRIBUTING.md) describes all requirements proposal needs to fulfill to reach `Stage 3: Accepted`, | ||
- Pull request must be: | ||
- labeled as an accepted proposal, | ||
- created against the feature branch, | ||
- created in all repositories specified in contribution guide, | ||
- At least one user listed in [CODEOWNERS](CODEOWNERS) must approve the pull request in all related repositories. | ||
|
||
#### Prerequisites for first feature merge | ||
|
||
- `spec` - Make sure all the official examples located in the repository use the new version of the specification, | ||
- `spec-json-schemas` - Make sure changes are introduced in the new JSON Schema file, and they are exposed [here](https://github.com/asyncapi/spec-json-schemas/blob/master/index.js), | ||
- `parser-js` - Make sure the list of supported AsyncAPI schema MIME types is extended with the new version [here](https://github.com/asyncapi/parser-js/blob/master/lib/asyncapiSchemaFormatParser.js#L43.) | ||
|
||
#### Merge | ||
|
||
- Merge can be done only by repository [CODEOWNERS](CODEOWNERS), | ||
- Every pull request must have a `feat: ` prefix that, after the merge, produces a release candidate with a minor version update. The major version must have `feat!: ` prefix, | ||
- Pull request in the `parser-js` can be merged only if it uses the release candidate of `@asyncapi/specs` package produced after merge of a pull request in `spec-json-schemas`, | ||
- First, changes are merged into the `spec` repository, then `spec-json-schemas` and then in `parser-js`. | ||
|
||
### Next steps | ||
|
||
- Update previously created draft of release notes with information about the new feature. Collaborate closely with feature contributors, | ||
- Notify maintainers of the following repositories that the first feature is merged and that release will be produced and therefore they need to start preparing for it: | ||
- [JavaScript Converter](https://github.com/asyncapi/converter-js/) | ||
- [Playground](https://github.com/asyncapi/playground/) | ||
- [React component](https://github.com/asyncapi/asyncapi-react/) | ||
- [Markdown template](https://github.com/asyncapi/markdown-template) | ||
|
||
### Steps only after first feature merge | ||
|
||
- Open draft pull requests in all repositories required by the contribution guide. They should point from the release branch in the upstream to the master branch in the upstream, | ||
- Update documentation in [website](https://github.com/asyncapi/website) to promote latest version of the specification. You can do it in the same pull request with release notes. No need to care for the specification markdown file on the Website. It updates automatically during work on release candidates and also after the final release. | ||
|
||
#### Communication | ||
|
||
Every release of the release candidate is automatically published on the AsyncAPI Twitter account and in the releases-dedicated Slack channel. Feel free to use other communication channels. Make sure that as many people as possible know about the change. Feel free to contact vendors upfront or other people that are interested in changes in the specification. | ||
|
||
### Ship it! | ||
|
||
The release is scheduled for a given month, not a specific day. You do not have to wait until the last day of the month with the release. We release often, so it can be released in June if the change will not make it in April. Most important is the quality and not quantity of the features. The quality feature means it is properly delivered in as many tools as possible with very good documentation and examples. | ||
|
||
- Release means merge of pull requests created from a release branch against the master branch. First, changes are merged into the `spec` repository, then `spec-json-schemas` and at the end in `parser-js`. Like in the case of the merge of release candidates, a pull request in `parser-js` can be merged only if it uses the final release of the new `@asyncapi/specs` package. | ||
- Make sure other maintainers from other projects under the AsyncAPI GitHub organization released their packages | ||
- Merge release notes and once they are published, link them under the GitHub release for the specification, like [here](https://github.com/asyncapi/spec/releases/tag/v2.1.0). | ||
- Make sure proper communication is sent from all the official AsyncAPI social accounts and on Slack | ||
|
||
Now relax as the next release is right behind the corner :smiley: |
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it feels weird that release notes would be in the blog. shouldn't they be in the repos for tools and on the website with the spec docs? ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Release notes for the website are there for a better presentation of the information and availability, like https://www.asyncapi.com/blog/release-notes-2.1.0. It is better to collaborate with other release contributors on a single PR
There is also always a release note in the GitHub release for a given tool, yup. There release here is for overall release summary, all features and thank you
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
okay perfect, just wanted to make sure release notes weren't only in the blog