Skip to content

Conversation

@lkerford
Copy link
Contributor

Removing the configuration option to modify the maximum variable length and changing it from 100 to 40.

40 is already a very long variable name, and we don't think that people will be changing this max length

Removing the configuration option to modify the maximum variable length and changing it from 100 to 40.

40 is already a very long variable name, and we don't think that people will be changing this max length
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some suggestions could not be made:

  • src/main/java/org/openrewrite/staticanalysis/ChainStringBuilderAppendCalls.java
    • lines 36-38

@timtebeek timtebeek added the enhancement New feature or request label Oct 22, 2024
@timtebeek
Copy link
Member

Feel free to merge whenever btw; We'll do a new release tomorrow.

@lkerford lkerford merged commit 619b623 into main Oct 22, 2024
2 checks passed
@lkerford lkerford deleted the set-max-variable-length-to-40 branch October 22, 2024 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants