-
-
Notifications
You must be signed in to change notification settings - Fork 11.1k
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
MAINT: Replace pavement.py #29456
Conversation
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]
8c164a1
to
3203001
Compare
Out of curiosity; did you vibe code this? And if so, how did you like it? |
No, it was mostly inherited from pavement.py, I just cleaned it up a bit. |
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>
1ccaab5
to
ba36756
Compare
ba36756
to
6bab2a5
Compare
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.
I'm not sure about the specifics, but the script itself looks nice and pretty to me :)
Replace
pavement.py
functionality with awrite_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.[skip azp] [skip cirrus] [skip actions]