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

Harden sbt-scalafix #270

Merged
merged 5 commits into from
Aug 8, 2017
Merged

Harden sbt-scalafix #270

merged 5 commits into from
Aug 8, 2017

Conversation

olafurpg
Copy link
Contributor

@olafurpg olafurpg commented Aug 8, 2017

This PR introduces several improvements to sbt-scalafix and scalafix-core/reflect/cli that popped up while running dynamically compiled rewrites in larger repos.

Previously, sbt-scalafix failed to parse rewrites like github:a/b.
Now, that input is both accepted and supported by tab completion.
- reuse Input.File by downloaded url.
- synchronize on rewrite compilation. Multi-module builds invoke
  scalafix in parallel on multiple threads. Scalac is not thread safe, so we
  need to make sure we only compile one source at a time.
- add scalafixVerbose setting to help debug issues.
- remove unused cats code in tests.
@olafurpg olafurpg merged commit 5406b4b into scalacenter:master Aug 8, 2017
@olafurpg olafurpg deleted the replace-syntax branch August 8, 2017 15:02
bjaglin pushed a commit to liancheng/scalafix that referenced this pull request May 23, 2023
Bumps [codecov/codecov-action](https://github.com/codecov/codecov-action) from 2.1.0 to 3.1.0.
- [Release notes](https://github.com/codecov/codecov-action/releases)
- [Changelog](https://github.com/codecov/codecov-action/blob/master/CHANGELOG.md)
- [Commits](codecov/codecov-action@v2.1.0...v3.1.0)

---
updated-dependencies:
- dependency-name: codecov/codecov-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

1 participant