Skip to content
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

Write directly to relevant notes file in changelog.py #19303

Merged
merged 10 commits into from
Jun 14, 2023

Conversation

huonw
Copy link
Contributor

@huonw huonw commented Jun 13, 2023

This updates changelog.py to write the (non-internal) changes to the relevant release notes file directly, rather than requiring a human to copy-paste it in. For now, the file is just mutated, without committing. The internal changes are still printed for the human to deal with.

For instance, pants run src/python/pants_release/changelog.py -- --prior 2.18.0.dev1 --new 2.18.0.dev2 adds a new section to src/python/pants/notes/2.18.x.md:

diff --git a/src/python/pants/notes/2.18.x.md b/src/python/pants/notes/2.18.x.md
index e648a4525c..d6668a24b1 100644
--- a/src/python/pants/notes/2.18.x.md
+++ b/src/python/pants/notes/2.18.x.md
@@ -1,5 +1,35 @@
 # 2.18.x Release Series
 
+## 2.18.0.dev2 (Jun 14, 2023)
+
+### New Features
+
+* Include complete platforms for FaaS environments for more reliable building ([#19253](https://github.com/pantsbuild/pants/pull/19253))
+
+* Add experimental support for Rustfmt ([#18842](https://github.com/pantsbuild/pants/pull/18842))
+
+* Helm deployment chart field ([#19234](https://github.com/pantsbuild/pants/pull/19234))
+
+### Plugin API Changes
+
+* Replace `include_special_cased_deps` flag with `should_traverse_deps_predicate` ([#19272](https://github.com/pantsbuild/pants/pull/19272))
+
+### Bug Fixes
+
+* Raise an error if isort can't read a config file ([#19294](https://github.com/pantsbuild/pants/pull/19294))
+
+* Improve handling of additional files in Helm unit tests ([#19263](https://github.com/pantsbuild/pants/pull/19263))
+
+* Add taplo to the release ([#19258](https://github.com/pantsbuild/pants/pull/19258))
+
+* Handle `from foo import *` wildcard imports in Rust dep inference parser ([#19249](https://github.com/pantsbuild/pants/pull/19249))
+
+* Support usage of `scala_artifact` addresses in `scalac_plugin` targets ([#19205](https://github.com/pantsbuild/pants/pull/19205))
+
+### Performance
+
+* `scandir` returns `Stat`s relative to its directory. ([#19246](https://github.com/pantsbuild/pants/pull/19246))
+
 ## 2.18.0.dev1 (Jun 02, 2023)
 
 ### New Features

This also moves it into the new pants_release root, adds some basic tests, and has it fetch the relevant branch directly, rather than prompting the user to do so. It also pulls out a git.py support module with helpers for exec-ing git as an external process.

The commits are individually reviewable.

This is a step towards automating more of the "start release" process, per #19279. After this core refactoring of the functionality, I think the next step is to combine it with the CONTRIBUTORS update and version bumping, and then after that string it all together on CI so that starting a release is fully automated.

@huonw huonw added the category:internal CI, fixes for not-yet-released features, etc. label Jun 13, 2023
@huonw huonw changed the title Have changelog.py write to the relevant file directly Write directly to relevant notes file in changelog.py Jun 13, 2023
Copy link
Sponsor Contributor

@benjyw benjyw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yay! I am currently working on code to generate the announcement message, including the list of contributors. So I will use your git() function once you merge.

@huonw
Copy link
Contributor Author

huonw commented Jun 14, 2023

Nice! I'll not touch build-support/bin/contributors.py for now, then.

@huonw huonw merged commit c3c652c into pantsbuild:main Jun 14, 2023
25 checks passed
@huonw huonw deleted the feature/changelog-edit branch June 14, 2023 06:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:internal CI, fixes for not-yet-released features, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants