Skip to content

Conversation

@robgruen
Copy link
Collaborator

No description provided.

@robgruen robgruen marked this pull request as ready for review December 31, 2025 23:35
Copy link
Collaborator

@gvanrossum gvanrossum left a comment

Choose a reason for hiding this comment

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

Approving so you can merge when you're satisfied.

@robgruen robgruen merged commit 496ffe2 into main Dec 31, 2025
2 of 3 checks passed
@robgruen robgruen deleted the dev/robgruen/workflow_gate2 branch December 31, 2025 23:56
@gvanrossum
Copy link
Collaborator

Now all we need is an outsider to push a test PR to verify that that doesn't run the tests. @bmerkle ?

@bmerkle
Copy link
Contributor

bmerkle commented Jan 1, 2026

Will do the test with the move of Tests to new directory in next PR once i am Home in a few hours..... Happy new year to you all 😃🍀👋🎉

@bmerkle
Copy link
Contributor

bmerkle commented Jan 1, 2026

ok PR #144 landed. @gvanrossum

gvanrossum pushed a commit that referenced this pull request Jan 1, 2026
- moved test and testdata to tests directory
- adopted make, config files and ci
- fixed failing tests and introducts tests directory variable

fix part2 for #125 
testing PR ci for #143
@gvanrossum
Copy link
Collaborator

The 3rd party experience is better, they get one failure of a job named permissions. (After an owner approves it.)

But as a project owner I still see 12 deployment approval emails?!

@robgruen
Copy link
Collaborator Author

robgruen commented Jan 2, 2026 via email

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.

4 participants