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.

Sign up for GitHub

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

Jump to bottom

Rewrites should be able to validate configuration during initialization #261

Closed
olafurpg opened this issue Jul 18, 2017 · 0 comments · Fixed by #316
Closed

Rewrites should be able to validate configuration during initialization #261

olafurpg opened this issue Jul 18, 2017 · 0 comments · Fixed by #316
Milestone
v0.5.0

Comments

@olafurpg
Copy link
Contributor

Currently, custom rewrites have to validate the configuration inside the def rewrite(ctx: RewriteCtx) method. It would be nice if they could validate the configuration only once during initialization.
@olafurpg olafurpg modified the milestone: v0.5 Aug 16, 2017
olafurpg added a commit to olafurpg/scalafix that referenced this issue Aug 30, 2017
@olafurpg
Add Rewrite.init, fixes scalacenter#261.
04daff3 
This method can be used to do any sort of preprocessing before fixing
files. For example, a rewrite that does dead-code elimination.could
build a call graph in `init`.
@olafurpg olafurpg mentioned this issue Aug 30, 2017
Merged
bjaglin pushed a commit to liancheng/scalafix that referenced this issue May 23, 2023
@scala-steward
Update scalafmt-core to 3.4.3 (scalacenter#261)
e26c06f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.5.0
Development

Successfully merging a pull request may close this issue.

Various changes combined into a single PR olafurpg/scalafix
1 participant
@olafurpg