Skip to content

Conversation

yedayak
Copy link
Collaborator

@yedayak yedayak commented Sep 18, 2025

No description provided.

@yedayak yedayak marked this pull request as draft September 18, 2025 13:00
@yedayak yedayak marked this pull request as ready for review September 18, 2025 15:01
Copy link
Owner

@scop scop left a comment

Choose a reason for hiding this comment

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

Interesting, TIL & LGTM.

I've had the intention to work on this area and other security related things, looks like this will blend in nicely. One related thing I just submitted is #1433

@yedayak
Copy link
Collaborator Author

yedayak commented Sep 18, 2025

One thing I will say about this is I would love to somehow test that releases will work correctly with this PR, is there a way to simulate the release please process somewhere?

@scop
Copy link
Owner

scop commented Sep 18, 2025

The first thing springing to mind is I guess you could tweak your own fork to do a release there. But as far as this PR is concerned, while that would be nice, I don't think that's required -- I hope I'll get to adding some other goodies which will require some eyeballing at release time anyway so I'll be aware.

@yedayak yedayak merged commit d6cfcf9 into scop:main Sep 18, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants