Skip to content

Conversation

@dependabot
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Dec 12, 2023

Bumps com.puppycrawl.tools:checkstyle from 8.19 to 8.29.

Release notes

Sourced from com.puppycrawl.tools:checkstyle's releases.

checkstyle-8.29

https://checkstyle.org/releasenotes.html#Release_8.29

checkstyle-8.28

https://checkstyle.org/releasenotes.html#Release_8.28

checkstyle-8.27

https://checkstyle.org/releasenotes.html#Release_8.27

checkstyle-8.26

https://checkstyle.org/releasenotes.html#Release_8.26

checkstyle-8.25

https://checkstyle.org/releasenotes.html#Release_8.25

checkstyle-8.24

https://checkstyle.org/releasenotes.html#Release_8.24

checkstyle-8.23

https://checkstyle.org/releasenotes.html#Release_8.23

checkstyle-8.22

https://checkstyle.org/releasenotes.html#Release_8.22

checkstyle-8.21

https://checkstyle.org/releasenotes.html#Release_8.21

checkstyle-8.20

https://checkstyle.org/releasenotes.html#Release_8.20

Commits
  • 8933d03 [maven-release-plugin] prepare release checkstyle-8.29
  • bd45909 Issue #7487: refactor code to use DetailAST.hasChildren()
  • 317e51f Issue #7487: add method hasChildren() to DetailAST
  • 89b4dcd Issue #3238: Java 8 Grammar: annotations on arrays and varargs
  • 252cd89 dependency: bump junit-pioneer from 0.5.1 to 0.5.2
  • 2ee2615 dependency: bump junit.version from 5.5.2 to 5.6.0
  • 4ed7cb8 minor: add space before xml comment end '-->' to ease reading and make links ...
  • c46a16d Issue #7468: disable 'external-parameter-entities' feature by default
  • dfed794 minor: add missing test case to SuperCloneCheckTest
  • 24e7bdf dependency: bump antlr4.version from 4.7.2 to 4.8-1
  • Additional commits viewable in compare view

Dependabot compatibility score

You can trigger a rebase of this PR by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
    You can disable automated security fix PRs for this repo from the Security Alerts page.

Note
Automatic rebases have been disabled on this pull request as it has been open for over 30 days.

Bumps [com.puppycrawl.tools:checkstyle](https://github.com/checkstyle/checkstyle) from 8.19 to 8.29.
- [Release notes](https://github.com/checkstyle/checkstyle/releases)
- [Commits](checkstyle/checkstyle@checkstyle-8.19...checkstyle-8.29)

---
updated-dependencies:
- dependency-name: com.puppycrawl.tools:checkstyle
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot requested a review from lidavidm as a code owner December 12, 2023 13:42
@dependabot dependabot bot added dependencies Dependabot-only java Dependabot-only labels Dec 12, 2023
@github-actions
Copy link

Thanks for opening a pull request!

If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose

Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project.

Then could you also rename the pull request title in the following format?

GH-${GITHUB_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

or

MINOR: [${COMPONENT}] ${SUMMARY}

In the case of PARQUET issues on JIRA the title also supports:

PARQUET-${JIRA_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

See also:

@kou kou changed the title Bump com.puppycrawl.tools:checkstyle from 8.19 to 8.29 in /java/maven MINOR: [Java] Bump com.puppycrawl.tools:checkstyle from 8.19 to 8.29 in /java/maven Dec 14, 2023
@vibhatha
Copy link
Contributor

should we close this?

@lidavidm
Copy link
Member

I think we should try to get this minor version bump in if possible. If it's a large effort I'd rather spend that switching to google-java-format instead.

@vibhatha
Copy link
Contributor

I can take a look.

@vibhatha
Copy link
Contributor

PR to replace this is created here: #39694
But still in the draft mode, testing.

lidavidm pushed a commit that referenced this pull request Feb 5, 2024
…29 (#39694)

### Rationale for this change

This PR was created in place of #39202 to integrate the `puppycrawl.tools.checkstyle` upgrade. 

### What changes are included in this PR?

Style changes in Java classes and core changes to the style format itself. 
Some unsupported attributes have been removed. And some attributes have
been reorganized upon the provided guidelines in the documentation. 

### Are these changes tested?

N/A 
Tested by existing checkstyle guideline. 

### Are there any user-facing changes?

No

* Closes: #39946

Lead-authored-by: Vibhatha Lakmal Abeykoon <vibhatha@gmail.com>
Co-authored-by: vibhatha <vibhatha@gmail.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
@vibhatha
Copy link
Contributor

vibhatha commented Feb 6, 2024

@lidavidm I think we could close this.

@lidavidm lidavidm closed this Feb 6, 2024
@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Feb 6, 2024

OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting @dependabot ignore this major version or @dependabot ignore this minor version. You can also ignore all major, minor, or patch releases for a dependency by adding an ignore condition with the desired update_types to your config file.

If you change your mind, just re-open this PR and I'll resolve any conflicts on it.

@lidavidm lidavidm deleted the dependabot/maven/java/maven/com.puppycrawl.tools-checkstyle-8.29 branch February 6, 2024 14:02
dgreiss pushed a commit to dgreiss/arrow that referenced this pull request Feb 19, 2024
… to 8.29 (apache#39694)

### Rationale for this change

This PR was created in place of apache#39202 to integrate the `puppycrawl.tools.checkstyle` upgrade. 

### What changes are included in this PR?

Style changes in Java classes and core changes to the style format itself. 
Some unsupported attributes have been removed. And some attributes have
been reorganized upon the provided guidelines in the documentation. 

### Are these changes tested?

N/A 
Tested by existing checkstyle guideline. 

### Are there any user-facing changes?

No

* Closes: apache#39946

Lead-authored-by: Vibhatha Lakmal Abeykoon <vibhatha@gmail.com>
Co-authored-by: vibhatha <vibhatha@gmail.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
kou pushed a commit to apache/arrow-java that referenced this pull request Nov 25, 2024
…29 (#39694)

### Rationale for this change

This PR was created in place of apache/arrow#39202 to integrate the `puppycrawl.tools.checkstyle` upgrade. 

### What changes are included in this PR?

Style changes in Java classes and core changes to the style format itself. 
Some unsupported attributes have been removed. And some attributes have
been reorganized upon the provided guidelines in the documentation. 

### Are these changes tested?

N/A 
Tested by existing checkstyle guideline. 

### Are there any user-facing changes?

No

* Closes: #39946

Lead-authored-by: Vibhatha Lakmal Abeykoon <vibhatha@gmail.com>
Co-authored-by: vibhatha <vibhatha@gmail.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
jmao-denver pushed a commit to jmao-denver/dh-flight-sql-jdbc that referenced this pull request Aug 5, 2025
…29 (#39694)

### Rationale for this change

This PR was created in place of apache/arrow#39202 to integrate the `puppycrawl.tools.checkstyle` upgrade. 

### What changes are included in this PR?

Style changes in Java classes and core changes to the style format itself. 
Some unsupported attributes have been removed. And some attributes have
been reorganized upon the provided guidelines in the documentation. 

### Are these changes tested?

N/A 
Tested by existing checkstyle guideline. 

### Are there any user-facing changes?

No

* Closes: #39946

Lead-authored-by: Vibhatha Lakmal Abeykoon <vibhatha@gmail.com>
Co-authored-by: vibhatha <vibhatha@gmail.com>
Signed-off-by: David Li <li.davidm96@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting review Awaiting review Component: Java dependencies Dependabot-only java Dependabot-only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants