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

Default value for when default_permissions argument is not provided #86

Merged
merged 4 commits into from
Feb 2, 2023

Conversation

Rictus
Copy link
Contributor

@Rictus Rictus commented Jan 30, 2023

No description provided.

@Rictus Rictus mentioned this pull request Jan 30, 2023
@Rictus
Copy link
Contributor Author

Rictus commented Jan 31, 2023

Hey @codemation I do not know what is wrong with the linter github action. Do you have any suggestion ?
Thanks!

Edit: It was related to the isort version

@codemation
Copy link
Owner

Hey @Rictus , good catch with isort, looks like you need to setup precommit locally and try to apply the change it found here via a new commit: https://github.com/codemation/easyauth/actions/runs/4058334348/jobs/6997735511#step:5:44

@Rictus
Copy link
Contributor Author

Rictus commented Feb 1, 2023

@codemation Should look good now 👍

@codemation codemation merged commit 4d4a2f4 into codemation:main Feb 2, 2023
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