Introducing package CHANGELOGs#131
Conversation
💚 Build Succeeded
Expand to view the summary
Build stats
Trends 🧪 |
mtojek
left a comment
There was a problem hiding this comment.
Nice! Looking forward to seeing this in action.
|
@mtojek Thanks for reviewing this PR but I thought of a couple of things after I put up this PR so I wanted to raise them here and get your opinions:
|
Most likely it's personal preference, but it's easier to find if there is a changelog package if it's in the root, close to the
I think it might be easier to keep the changelog file additive and don't clean it before releases. Actually it might be convenient not to jump over versions to check when a feature has been released. |
9d860eb to
ebe7262
Compare
|
Thanks for the feedback, @mtojek. Based on it, I'm going with the changes in this PR as-is. |
|
Thanks for getting this in. Looking forward to see this showing up in the packages. |
* Remove details * Fixing more typos
What does this PR do?
This PR enhances the package spec to require
CHANGELOG.ymlfiles at the root folder of every package.Why is it important?
So package consumers can track changes to packages across versions.
Checklist
code/go/internal/validator/test/packagesthat prove my change is effective.versions/N/changelog.yml.Related issues