Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Infer scalafmt version from .scalafmt.conf instead of using arg #5

Merged
merged 2 commits into from
Jun 24, 2024

Conversation

lzm0
Copy link

@lzm0 lzm0 commented Jun 24, 2024

The version field in Scalafmt is mandatory, so passing it as an action argument is unnecessary.

Since v3.1.0, the version parameter is required to be specified explicitly.

The action will always read the version field from .scalafmt.conf instead.

@lzm0 lzm0 force-pushed the remove-version-arg branch from 8a13f3e to 42a4159 Compare June 24, 2024 16:14
@lzm0 lzm0 force-pushed the remove-version-arg branch from 42a4159 to f18aabe Compare June 24, 2024 16:14
@lzm0 lzm0 merged commit 9d1eebd into main Jun 24, 2024
1 check passed
@lzm0 lzm0 deleted the remove-version-arg branch June 24, 2024 17:17
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants