Skip to content
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

hooks: add hook for langdetect and swagger_spec_validator #302

Merged
merged 3 commits into from
Aug 24, 2021
Merged

hooks: add hook for langdetect and swagger_spec_validator #302

merged 3 commits into from
Aug 24, 2021

Conversation

eric15342335
Copy link
Contributor

#285 and #296
I seems can not run the workflow
sshot-001

@eric15342335 eric15342335 requested review from a team and Legorooj and removed request for a team August 24, 2021 05:06
@Legorooj Legorooj requested review from bwoodsend and removed request for Legorooj August 24, 2021 06:23
@bwoodsend
Copy link
Member

I seems can not run the workflow

This is rather odd. The actions for both the master repo and my fork are behaving normally so it appears to be specific to your fork but your master branch is even with ours so I don't see what the difference could be. In eric15342335/pyinstaller-hooks-contrib/settings/actions, do you have Allow all actions selected? That's what we have.

Copy link
Member

@bwoodsend bwoodsend left a comment

Choose a reason for hiding this comment

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

We need news entries for each one. Use the issue numbers instead of the PR numbers in the news entries' filenames.

@bwoodsend
Copy link
Member

I'd prefer that each news entry was added in the same commit as the hook and test rather than an extra add news item commit but it's not that important. I've triggered CI from my own fork for now. https://github.com/bwoodsend/pyinstaller-hooks-contrib/actions/runs/1162353622

@bwoodsend bwoodsend merged commit ae5a75c into pyinstaller:master Aug 24, 2021
@bwoodsend
Copy link
Member

@eric15342335 I'd like to add you to the PyInstaller team and give you basic triaging permissions. Basically you'd be able to do what I do here - make sure new pull requests make sense and have tests and news entries. Then I'll eventually give you push access so that you can start merging yourself too. You'll also be able to do the same on the main PyInstaller repo too. Sound OK?

@eric15342335
Copy link
Contributor Author

Sounds pretty good, thank you!

@eric15342335 eric15342335 deleted the hook-langdetect branch August 24, 2021 11:52
@bwoodsend
Copy link
Member

OK, let me work out who has permission to grant other people permissions...

@bwoodsend
Copy link
Member

In theory, you should have received an invite. Could you also send me an email so that I can send you a link to the Discord channel we use to talk to each other.

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.

None yet

2 participants