Skip to content

Update scalafmt-core to 3.9.7 #391

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

scala-steward
Copy link
Contributor

@scala-steward scala-steward commented Jun 1, 2025

About this PR

📦 Updates org.scalameta:scalafmt-core from 3.9.6 to 3.9.7

📜 GitHub Release Notes - Version Diff

Usage

Please merge!

I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.

If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.

Configure Scala Steward for your repository with a .scala-steward.conf file.

Have a fantastic day writing Scala!

⚙ Adjust future updates

Add this to your .scala-steward.conf file to ignore future updates of this dependency:

updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

Or, add this to slow down future updates of this dependency:

dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalameta", artifactId = "scalafmt-core" }
}]
labels: library-update, early-semver-patch, semver-spec-patch, commit-count:n:3

Summary by CodeRabbit

  • Chores
    • Updated code formatting and alignment throughout the project for improved consistency and readability.
    • Upgraded scalafmt configuration to version 3.9.7.
    • Added a file to ignore specific commits in git blame operations.
  • Style
    • Adjusted whitespace and alignment in source and test files without affecting functionality or logic.

Copy link

coderabbitai bot commented Jun 1, 2025

Walkthrough

This update introduces a .git-blame-ignore-revs file to exclude a scalafmt reformat commit from blame history, upgrades the scalafmt configuration to version 3.9.7, and applies consistent whitespace and alignment adjustments across source and test files. No logic, control flow, or public API changes are present; all modifications are purely stylistic.

Changes

File(s) Change Summary
.git-blame-ignore-revs Added file to ignore a specific scalafmt reformat commit in git blame.
.scalafmt.conf Updated scalafmt version from 3.9.6 to 3.9.7.
build.sbt Reformatted assignment lines and aligned settings for consistency.
cats-effect/src/main/scala/io/kaizensolutions/virgil/internal/CQLExecutorImpl.scala Adjusted whitespace for alignment in variable assignment.
cats-effect/src/test/scala/io/kaizensolutions/virgil/CQLExecutorSpec.scala Reformatted variable assignments for alignment in tests.
cats-effect/src/test/scala/io/kaizensolutions/virgil/CassandraContainer.scala Adjusted whitespace and alignment in variable declarations.
cats-effect/src/test/scala/io/kaizensolutions/virgil/DeleteBuilderSpec.scala Reformatted variable assignments for alignment in test case.
cats-effect/src/test/scala/io/kaizensolutions/virgil/InsertBuilderSpec.scala Reformatted variable assignments for alignment in test case.
cats-effect/src/test/scala/io/kaizensolutions/virgil/ResourceSuite.scala Adjusted alignment and whitespace in implicit vals and for-comprehensions.
cats-effect/src/test/scala/io/kaizensolutions/virgil/UpdateBuilderSpec.scala Reformatted variable assignments for alignment in test cases.
core/src/main/scala/io/kaizensolutions/virgil/CQL.scala Adjusted whitespace for alignment in a local variable assignment.
core/src/main/scala/io/kaizensolutions/virgil/codecs/CqlPrimitiveDecoder.scala Reformatted variable and method assignments for alignment.
core/src/main/scala/io/kaizensolutions/virgil/codecs/CqlPrimitiveEncoder.scala Reformatted method definitions for alignment in multiple case classes/objects.
core/src/main/scala/io/kaizensolutions/virgil/dsl/InsertBuilder.scala Reformatted variable assignment for alignment.
core/src/main/scala/io/kaizensolutions/virgil/internal/CqlStatementRenderer.scala Adjusted whitespace for alignment in local variable assignment.
core/src/test/scala/io/kaizensolutions/virgil/CqlInterpolatorSpec.scala Reformatted assignment of a string variable for alignment in test.
core/src/test/scala/io/kaizensolutions/virgil/models/CollectionsSpecDatatypes.scala Adjusted whitespace for alignment in a value declaration.
zio/src/main/scala/io/kaizensolutions/virgil/internal/CQLExecutorImpl.scala Adjusted whitespace for alignment in variable assignment.
zio/src/test/scala/io/kaizensolutions/virgil/AllTests.scala Adjusted whitespace for alignment in for-comprehension bindings.
zio/src/test/scala/io/kaizensolutions/virgil/CQLExecutorSpec.scala Reformatted variable assignment for alignment in a test case.
zio/src/test/scala/io/kaizensolutions/virgil/CassandraContainer.scala Adjusted whitespace and alignment in variable declarations.
zio/src/test/scala/io/kaizensolutions/virgil/DeleteBuilderSpec.scala Adjusted indentation for alignment in test case.
zio/src/test/scala/io/kaizensolutions/virgil/RelationSpec.scala Added extra space for alignment in a variable assignment.
zio/src/test/scala/io/kaizensolutions/virgil/SelectBuilderSpec.scala Adjusted whitespace for alignment in variable declarations in test cases.
zio/src/test/scala/io/kaizensolutions/virgil/UpdateBuilderSpec.scala Reformatted variable assignments for alignment in test cases.

Poem

In the warren, code aligns just right,
Spaces and equals, now a tidy sight.
With scalafmt’s latest, we hop along,
Blame won’t trip on what’s not wrong.
A carrot for style, a hop for cheer—
Our code’s neat and ready,
Let’s celebrate here!

((\
( -.-)
o_(")(")


📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4e18318 and b9d753e.

📒 Files selected for processing (25)
  • .git-blame-ignore-revs (1 hunks)
  • .scalafmt.conf (1 hunks)
  • build.sbt (2 hunks)
  • cats-effect/src/main/scala/io/kaizensolutions/virgil/internal/CQLExecutorImpl.scala (1 hunks)
  • cats-effect/src/test/scala/io/kaizensolutions/virgil/CQLExecutorSpec.scala (3 hunks)
  • cats-effect/src/test/scala/io/kaizensolutions/virgil/CassandraContainer.scala (1 hunks)
  • cats-effect/src/test/scala/io/kaizensolutions/virgil/DeleteBuilderSpec.scala (1 hunks)
  • cats-effect/src/test/scala/io/kaizensolutions/virgil/InsertBuilderSpec.scala (1 hunks)
  • cats-effect/src/test/scala/io/kaizensolutions/virgil/ResourceSuite.scala (3 hunks)
  • cats-effect/src/test/scala/io/kaizensolutions/virgil/UpdateBuilderSpec.scala (2 hunks)
  • core/src/main/scala/io/kaizensolutions/virgil/CQL.scala (1 hunks)
  • core/src/main/scala/io/kaizensolutions/virgil/codecs/CqlPrimitiveDecoder.scala (4 hunks)
  • core/src/main/scala/io/kaizensolutions/virgil/codecs/CqlPrimitiveEncoder.scala (6 hunks)
  • core/src/main/scala/io/kaizensolutions/virgil/dsl/InsertBuilder.scala (1 hunks)
  • core/src/main/scala/io/kaizensolutions/virgil/internal/CqlStatementRenderer.scala (1 hunks)
  • core/src/test/scala/io/kaizensolutions/virgil/CqlInterpolatorSpec.scala (1 hunks)
  • core/src/test/scala/io/kaizensolutions/virgil/models/CollectionsSpecDatatypes.scala (1 hunks)
  • zio/src/main/scala/io/kaizensolutions/virgil/internal/CQLExecutorImpl.scala (1 hunks)
  • zio/src/test/scala/io/kaizensolutions/virgil/AllTests.scala (2 hunks)
  • zio/src/test/scala/io/kaizensolutions/virgil/CQLExecutorSpec.scala (1 hunks)
  • zio/src/test/scala/io/kaizensolutions/virgil/CassandraContainer.scala (1 hunks)
  • zio/src/test/scala/io/kaizensolutions/virgil/DeleteBuilderSpec.scala (1 hunks)
  • zio/src/test/scala/io/kaizensolutions/virgil/RelationSpec.scala (1 hunks)
  • zio/src/test/scala/io/kaizensolutions/virgil/SelectBuilderSpec.scala (3 hunks)
  • zio/src/test/scala/io/kaizensolutions/virgil/UpdateBuilderSpec.scala (2 hunks)
🧰 Additional context used
🧬 Code Graph Analysis (4)
zio/src/test/scala/io/kaizensolutions/virgil/DeleteBuilderSpec.scala (6)
core/src/main/scala/io/kaizensolutions/virgil/CQL.scala (2)
  • truncate (168-171)
  • insert (136-145)
core/src/main/scala/io/kaizensolutions/virgil/internal/CqlStatementRenderer.scala (1)
  • truncate (236-239)
core/src/test/scala/io/kaizensolutions/virgil/models/CursorSpecDatatypes.scala (2)
  • truncate (34-34)
  • insert (36-47)
zio/src/main/scala/io/kaizensolutions/virgil/internal/CQLExecutorImpl.scala (1)
  • execute (24-42)
zio/src/main/scala/io/kaizensolutions/virgil/CQLExecutor.scala (1)
  • execute (25-28)
core/src/test/scala/io/kaizensolutions/virgil/models/DeleteBuilderSpecDatatypes.scala (2)
  • insert (32-42)
  • find (25-32)
cats-effect/src/test/scala/io/kaizensolutions/virgil/ResourceSuite.scala (4)
core/src/test/scala/io/kaizensolutions/virgil/models/UserDefinedTypesSpecDatatypes.scala (1)
  • UserDefinedTypesSpecDatatypes (16-266)
core/src/test/scala/io/kaizensolutions/virgil/models/UpdateBuilderSpecDatatypes.scala (1)
  • UpdateBuilderSpecDatatypes (8-65)
core/src/test/scala/io/kaizensolutions/virgil/models/RelationSpecDatatypes.scala (1)
  • RelationSpecDatatypes (9-46)
cats-effect/src/main/scala/io/kaizensolutions/virgil/CQLExecutor.scala (1)
  • CQLExecutor (23-44)
core/src/main/scala/io/kaizensolutions/virgil/codecs/CqlPrimitiveEncoder.scala (1)
core/src/main/scala/io/kaizensolutions/virgil/codecs/CqlPrimitiveDecoder.scala (16)
  • driverClass (244-244)
  • driverClass (252-252)
  • driverClass (260-260)
  • driverClass (268-268)
  • driverClass (276-276)
  • driverClass (284-284)
  • driverClass (292-292)
  • driverClass (300-300)
  • driverClass (308-308)
  • driverClass (316-316)
  • driverClass (324-324)
  • driverClass (332-332)
  • driverClass (340-340)
  • driverClass (348-348)
  • driverClass (356-356)
  • driverClass (364-364)
core/src/main/scala/io/kaizensolutions/virgil/internal/CqlStatementRenderer.scala (1)
core/src/main/scala/io/kaizensolutions/virgil/internal/BindMarker.scala (1)
  • empty (65-65)
⏰ Context from checks skipped due to timeout of 90000ms (7)
  • GitHub Check: Build and Test (ubuntu-latest, 2.13.16, temurin@21)
  • GitHub Check: Build and Test (ubuntu-latest, 3.3.6, temurin@17)
  • GitHub Check: Build and Test (ubuntu-latest, 3.3.6, temurin@21)
  • GitHub Check: Build and Test (ubuntu-latest, 2.12.20, temurin@17)
  • GitHub Check: Build and Test (ubuntu-latest, 2.13.16, temurin@17)
  • GitHub Check: Build and Test (ubuntu-latest, 2.12.20, temurin@21)
  • GitHub Check: Summary
🔇 Additional comments (34)
core/src/test/scala/io/kaizensolutions/virgil/models/CollectionsSpecDatatypes.scala (1)

105-105: Formatting-only alignment update. Adjusted whitespace for the tableName declaration to conform to the updated scalafmt rules. No functional change introduced.

zio/src/test/scala/io/kaizensolutions/virgil/RelationSpec.scala (1)

36-37: Stylistic alignment of val update declaration.
This whitespace-only change improves vertical alignment of the equals sign for readability without affecting behavior.

core/src/main/scala/io/kaizensolutions/virgil/CQL.scala (1)

79-79: Align whitespace for _ binding.
Purely formatting: adds spaces before = to align with surrounding code. No semantic impact.

cats-effect/src/test/scala/io/kaizensolutions/virgil/UpdateBuilderSpec.scala (2)

46-47: Align val update assignment.
Whitespace-only formatting to vertically align the = operator. Behavior remains unchanged.


90-91: Formatting: align equals for val update.
Improves consistency of spacing around =; no functional change.

zio/src/test/scala/io/kaizensolutions/virgil/UpdateBuilderSpec.scala (2)

31-32: Consistent alignment of val update.
Whitespace-only change to align the equals sign in the declaration. No logic change.


70-71: Formatting tweak: align val update.
Adds spaces for visual consistency; behavior unaffected.

cats-effect/src/test/scala/io/kaizensolutions/virgil/InsertBuilderSpec.scala (1)

26-27: Align <- operators in for-comprehension.
Whitespace-only adjustment to improve alignment of binding arrows. No semantic impact.

core/src/test/scala/io/kaizensolutions/virgil/CqlInterpolatorSpec.scala (1)

71-71: Align expected string assignment
The realigned val expected improves consistency with other multiline string definitions and enhances readability.

zio/src/test/scala/io/kaizensolutions/virgil/SelectBuilderSpec.scala (3)

35-35: Align variable declaration indentation
Adjusted the indentation of the artist declaration to match the suite’s formatting style.


54-55: Consistent alignment for query variables
Realigned artist and album declarations for uniform readability in this test.


75-77: Improve alignment of test variables
Aligned artist, album, and song declarations to maintain consistent indentation across tests.

zio/src/test/scala/io/kaizensolutions/virgil/CQLExecutorSpec.scala (1)

110-110: Align truncateData assignment
The spacing before the equals sign is adjusted for better alignment with surrounding code.

core/src/main/scala/io/kaizensolutions/virgil/dsl/InsertBuilder.scala (1)

25-25: Align allColumns assignment
Enhanced readability by aligning the = operator with other variable declarations in this method.

zio/src/test/scala/io/kaizensolutions/virgil/DeleteBuilderSpec.scala (1)

29-31: Consistent indentation for setup steps
Incremented indentation for the truncate and insert calls to match the block’s style.

core/src/main/scala/io/kaizensolutions/virgil/codecs/CqlPrimitiveEncoder.scala (1)

299-299: LGTM! Consistent alignment improvements.

The vertical alignment of driverClass method definitions improves code readability and maintains consistency across all primitive encoder implementations. These formatting changes are well-applied by scalafmt 3.9.7.

Also applies to: 315-315, 335-335, 351-351, 371-371, 386-386

core/src/main/scala/io/kaizensolutions/virgil/codecs/CqlPrimitiveDecoder.scala (2)

83-83: Improved variable assignment alignment.

The vertical alignment of the error variable assignments enhances code readability and maintains consistency in the error handling sections.

Also applies to: 169-169


400-400: Consistent driverClass method alignment.

The formatting changes align the driverClass method definitions consistently with the encoder implementations, improving overall codebase uniformity.

Also applies to: 478-478

.scalafmt.conf (1)

1-1: Appropriate scalafmt version update.

The update from version 3.9.6 to 3.9.7 is a standard patch release that typically includes bug fixes and minor improvements. This maintenance update helps keep the formatting toolchain current and consistent.

.git-blame-ignore-revs (1)

1-2: Excellent practice: Git blame ignore file added.

Adding the .git-blame-ignore-revs file is a best practice that excludes formatting-only commits from git blame operations. This makes it easier to trace the actual functional changes in the codebase history while maintaining clean formatting. The descriptive comment clearly indicates the commit's purpose.

core/src/main/scala/io/kaizensolutions/virgil/internal/CqlStatementRenderer.scala (1)

298-298: Consistent formatting of initial tuple assignment
The realignment of the initial variable enhances readability without affecting any logic.

zio/src/test/scala/io/kaizensolutions/virgil/CassandraContainer.scala (1)

14-14: Align environment variable declarations
The spacing adjustments for datastaxEnv and vanilla improve visual consistency. No functional changes.

Also applies to: 18-18

zio/src/test/scala/io/kaizensolutions/virgil/AllTests.scala (1)

32-32: Re-align <- bindings for readability
These whitespace tweaks bring the two ZIO bindings into alignment and do not alter any behavior.

Also applies to: 54-54

cats-effect/src/test/scala/io/kaizensolutions/virgil/CassandraContainer.scala (1)

17-17: Whitespace alignment in container setup
The updated spacing on datastaxEnv and vanilla declarations is purely cosmetic.

Also applies to: 21-21

cats-effect/src/main/scala/io/kaizensolutions/virgil/internal/CQLExecutorImpl.scala (1)

159-159: Standardize indentation for initial builder
This change aligns the assignment for better visual consistency, with no logic impact.

zio/src/main/scala/io/kaizensolutions/virgil/internal/CQLExecutorImpl.scala (1)

174-174: LGTM! Good alignment improvement.

The extra spacing before the assignment operator improves visual alignment and readability without affecting any functionality.

build.sbt (1)

7-8: LGTM! Consistent alignment improvements.

The spacing adjustments around assignment operators improve visual consistency and readability throughout the build configuration without affecting any build logic or settings.

Also applies to: 55-55

cats-effect/src/test/scala/io/kaizensolutions/virgil/DeleteBuilderSpec.scala (1)

44-46: LGTM! Improved indentation consistency.

The indentation adjustments improve visual alignment within the test method without affecting the test logic or assertions.

cats-effect/src/test/scala/io/kaizensolutions/virgil/CQLExecutorSpec.scala (1)

21-21: LGTM! Enhanced alignment and readability.

The spacing adjustments improve visual consistency in variable declarations and assignments throughout the test file without affecting any test logic or functionality.

Also applies to: 122-122, 160-160

cats-effect/src/test/scala/io/kaizensolutions/virgil/ResourceSuite.scala (5)

18-19: Formatting-only change: alignment of implicit Show and ResourceTag instances
These lines have been re-indented to align with the updated Scalafmt 3.9.7 rules. No functional modifications have been made.


22-22: Formatting-only change: alignment of UpdateBuilderSpecPersonShow
Whitespace adjustments only, matching the project’s new formatting conventions.


25-25: Formatting-only change: alignment of RelationSpecPersonShow
This is a pure whitespace realignment; behavior is unchanged.


53-53: Formatting-only change: spacing before <- in the for comprehension
Adjusted indentation to conform with the updated formatting profile. No logic changes.


70-71: Formatting-only change: alignment of resource bindings in SharedResources
Spaces were realigned around the <- operators for consistency under Scalafmt 3.9.7. No functional impact.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Explain this complex logic.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai explain this code block.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and explain its main purpose.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai generate docstrings to generate docstrings for this PR.
  • @coderabbitai generate sequence diagram to generate a sequence diagram of the changes in this PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

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