-
Notifications
You must be signed in to change notification settings - Fork 24
release-notes: Initial import for all streams #477
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
Conversation
Replaces coreos/fedora-coreos-tracker#1124 |
Might need to add a top object like |
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.
Generally LGTM - a few suggestions.
S3 syncing in coreos/fedora-coreos-pipeline#501. |
👍🏻 to everything here. Will update. |
Updated from suggestions. |
Conversion to JSON gives me:
|
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.
LGTM!
Updated and fixed. Should be good to merge now. |
Import a set of YAML documents that list issues fixed in a given Fedora CoreOS release. This list will be displayed as part of the release notes at https://getfedora.org/en/coreos See: coreos/fedora-coreos-tracker#194
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.
In a follow-up, can we codify in the checklists the steps that we've agreed upon for the manual way of doing things for now?
Need a review from you @dustymabe as I need 2 here. |
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.
LGTM
Import a set of YAML documents that list issues fixed in a given Fedora
CoreOS release.
This list will be displayed as part of the release notes at
https://getfedora.org/en/coreos
See: coreos/fedora-coreos-tracker#194