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

Lints: update to setup Python v4 #3243

Merged
merged 3 commits into from
Mar 14, 2023
Merged

Lints: update to setup Python v4 #3243

merged 3 commits into from
Mar 14, 2023

Conversation

SpencerIsGiddy
Copy link
Contributor

@SpencerIsGiddy SpencerIsGiddy commented Jan 21, 2023

Updates setup python to v4 in lints.yml

@SpencerIsGiddy SpencerIsGiddy changed the title Update lints.yml Lints: update to python 3.11 and setup python v4 Jan 21, 2023
@SpencerIsGiddy SpencerIsGiddy marked this pull request as ready for review January 21, 2023 03:53
@raidensakura
Copy link
Member

raidensakura commented Jan 21, 2023

I don't think inclusion of 3.11 is necessary, Modmail does not support it yet so there's no need to include it in action tests, it's only going to extend runtime.

@SpencerIsGiddy SpencerIsGiddy changed the title Lints: update to python 3.11 and setup python v4 Lints: update to setup Python v4 Jan 21, 2023
@SpencerIsGiddy
Copy link
Contributor Author

I don't think inclusion of 3.11 is necessary, Modmail does not support it yet so there's no need to include it in action tests, it's only going to extend runtime.

Ok. I reverted the change

@Taaku18 Taaku18 merged commit 960a36e into modmail-dev:master Mar 14, 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.

3 participants