Skip to content

MAINT: Replace pavement.py #29456

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

Merged
merged 3 commits into from
Jul 28, 2025
Merged

MAINT: Replace pavement.py #29456

merged 3 commits into from
Jul 28, 2025

Conversation

charris
Copy link
Member

@charris charris commented Jul 28, 2025

Replace pavement.py functionality with a write_release.py script. This gets rid of a paver dependency and simplifies the release process. This can be the basis of a new spin command.

  • Add tools/write_release.py
  • Remove pavement.py

[skip azp] [skip cirrus] [skip actions]

Replace `pavement.py` functionality with a `write_release.py` script.
This gets rid of a paver dependency and simplifies the release process.

- Add tools/write_release.py
- Remove pavement.py

[skip azp] [skip cirrus] [skip actions]
@charris charris force-pushed the remove-pavement.py branch from 8c164a1 to 3203001 Compare July 28, 2025 15:27
@jorenham
Copy link
Member

Out of curiosity; did you vibe code this? And if so, how did you like it?

@charris
Copy link
Member Author

charris commented Jul 28, 2025

did you vibe code this?

No, it was mostly inherited from pavement.py, I just cleaned it up a bit.

@charris
Copy link
Member Author

charris commented Jul 28, 2025

Thanks for the suggestions, Joren, the Path object looks like it has a lot of cleanup potential.

Co-authored-by: Joren Hammudoglu <jhammudoglu@gmail.com>
@charris charris force-pushed the remove-pavement.py branch from 1ccaab5 to ba36756 Compare July 28, 2025 18:02
@charris charris force-pushed the remove-pavement.py branch from ba36756 to 6bab2a5 Compare July 28, 2025 18:08
Copy link
Member

@jorenham jorenham left a comment

Choose a reason for hiding this comment

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

I'm not sure about the specifics, but the script itself looks nice and pretty to me :)

@charris charris merged commit f41d553 into numpy:main Jul 28, 2025
76 checks passed
@charris charris deleted the remove-pavement.py branch July 28, 2025 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants