Skip to content

Conversation

@asottile
Copy link
Member

@asottile asottile commented Jun 2, 2020

Resolves #3502

@asottile
Copy link
Member Author

asottile commented Jun 2, 2020

note: this will change the job name so we'll need to adjust this in the repository settings

@nicoddemus
Copy link
Member

IMOHO this is not really necessary... linting is always one of the first jobs to finish, as far as I see, since we moved to GitHub actions.

@asottile
Copy link
Member Author

asottile commented Jun 2, 2020

IMOHO this is not really necessary... linting is always one of the first jobs to finish, as far as I see, since we moved to GitHub actions.

finishing quickly can open up another parallel runner. also saves some bandwidth / flakiness with clones 🤷

@nicoddemus
Copy link
Member

Yeah you got a point. 👍

@nicoddemus nicoddemus merged commit c871d8b into pytest-dev:master Jun 9, 2020
@asottile asottile deleted the cache_pre_commit branch June 9, 2020 15:58
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.

optimize travis builds

2 participants