Skip to content

Commit

Permalink
Move release notes to docs/notes, don't do wheel builds when changed (#…
Browse files Browse the repository at this point in the history
…20847)

As far as I can tell the release notes (formerly
`src/python/pants/notes`) are no longer included in wheel builds, so
they can be considered more "normal" documentation. This moves them, and
stops the classify-changes script from flagging them as a
`release`-level change; they'll now just be `docs`.

The pantsbuild.org sync_docs workflow copies the `docs/docs` directory,
so won't be automatically including these... but maybe
pantsbuild/pantsbuild.org#184 will change this
(that PR needs changes, after this).
  • Loading branch information
huonw authored Apr 27, 2024
1 parent 164f322 commit 2a72e0f
Show file tree
Hide file tree
Showing 58 changed files with 6 additions and 8 deletions.
4 changes: 1 addition & 3 deletions build-support/bin/classify_changed_files.py
Original file line number Diff line number Diff line change
Expand Up @@ -30,12 +30,10 @@ class Affected(enum.Enum):
"build-support/bin/rust/*",
]
_release_globs = [
# Any changes to these files should trigger wheel building. Notes too, as they are included in
# the wheel.
# Any changes to these files should trigger wheel building.
"pants.toml",
"src/python/pants/VERSION",
"src/python/pants/init/BUILD",
"src/python/pants/notes/*",
"src/python/pants_release/release.py",
]
_ci_config_globs = [
Expand Down
4 changes: 2 additions & 2 deletions build-support/bin/classify_changed_files_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,8 @@
{Affected.docs},
],
[
["src/python/pants/notes/2.16.x.md"],
{Affected.docs, Affected.release},
["docs/notes/2.16.x.md"],
{Affected.docs},
],
[["src/rust/engine/path/to/file.rs"], {Affected.rust}],
[["src/python/pants/VERSION"], {Affected.release}],
Expand Down
2 changes: 1 addition & 1 deletion docs/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@

This directory holds our handwritten documentation.

During a release, the contents of this directory is synced with our website repo (https://github.com/pantsbuild/pantsbuild.org) to be deployed.
During a release, the contents of the `docs/` subdirectory is synced with our website repo (https://github.com/pantsbuild/pantsbuild.org) to be deployed.
2 changes: 1 addition & 1 deletion docs/docs/contributions/releases/release-process.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ The release commit is the commit that bumps the VERSION string. For `dev`/`a0` r

If this is the first dev release in a new series:

1. Create a new file in ` src/python/pants/notes`, e.g. create `src/python/pants/notes/2.9.x.md`.
1. Create a new file in `docs/notes`, e.g. create `docs/notes/2.9.x.md`.
1. Copy the title and template over from the prior release, e.g. `2.8.x.md`.

### Bump the VERSION
Expand Down
2 changes: 1 addition & 1 deletion docs/docs/using-pants/restricted-internet-access.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,7 @@ url_template = "https://my.custom.host/bin/protoc/{version}/{platform}/protoc.zi

For simplicity, we used the original value for `url_platform_mapping`, meaning that we set up our hosted URL to store the macOS x86 binary at `.../osx-x86_64/protoc.zip`, for example. You can override the option `url_platform_mapping` if you want to use different values.

Occasionally, new Pants releases will upgrade to new versions of these binaries, which will be mentioned in the "User API Changes" part of the changelog [https://github.com/pantsbuild/pants/tree/master/src/python/pants/notes](https://github.com/pantsbuild/pants/tree/master/src/python/pants/notes). When upgrading to these new Pants releases, you should download the new artifact and upload a copy to your host.
Occasionally, new Pants releases will upgrade to new versions of these binaries, which will be mentioned in the "User API Changes" part of the changelog [https://github.com/pantsbuild/pants/tree/main/docs/notes](https://github.com/pantsbuild/pants/tree/main/docs/notes). When upgrading to these new Pants releases, you should download the new artifact and upload a copy to your host.

:::note Asking for help
It's possible that Pants does not yet have all the mechanisms it'll need to work with your organization's specific networking setup, which we'd love to fix.
Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.

0 comments on commit 2a72e0f

Please sign in to comment.