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

Enable --experimental-string-processing on most primer projects #2184

Merged
merged 2 commits into from
May 5, 2021

Conversation

cooperlees
Copy link
Collaborator

@cooperlees cooperlees commented May 3, 2021

  • We want to make this default so need to test it more
  • Fixed splat/star bug in extending black args for each project

Helps pave way for #2188

@JelleZijlstra JelleZijlstra added the skip news Pull requests that don't need a changelog entry. label May 3, 2021
@cooperlees cooperlees changed the title Enable --experimental-string-processing on all primer projects Enable --experimental-string-processing on most primer projects May 3, 2021
@cooperlees
Copy link
Collaborator Author

Hmm - Primer didn't trigger ... no idea why ...

@ichard26
Copy link
Collaborator

ichard26 commented May 3, 2021

Hmm - Primer didn't trigger ... no idea why ...

I thought I had let you know I manually disabed the primer workflow since I was iterating a LOT on that travis CI migration PR. Check the group chat again. I assumed this PR was done, but that was unreasonable. Only the fuzz workflow is disabled right now. Apologies for the disruption!

@cooperlees
Copy link
Collaborator Author

I don't know how to retrigger the Primer action now :| ... But it should pass ...

cooperlees added 2 commits May 5, 2021 07:41
- We want to make this default so need to test it more
- Fixed splat/star bug in extending black args for each project
@cooperlees cooperlees merged commit 07c8812 into master May 5, 2021
@cooperlees cooperlees deleted the primer_strings branch May 5, 2021 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip news Pull requests that don't need a changelog entry.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants