Skip to content

Pull #18610: Align rewrite readme batch #18589#18610

Closed
Pankraz76 wants to merge 1 commit into
checkstyle:masterfrom
Pankraz76:fix-align-ci
Closed

Pull #18610: Align rewrite readme batch #18589#18610
Pankraz76 wants to merge 1 commit into
checkstyle:masterfrom
Pankraz76:fix-align-ci

Conversation

@Pankraz76

@Pankraz76 Pankraz76 commented Jan 11, 2026

Copy link
Copy Markdown

Pull #18610: Align rewrite readme batch #18589

Checkstyle - Java Code Quality Tool

Checkstyle is a tool that ensures adherence to a code standard, or a set of best practices.


Continuous Integration: 🏭

Quality Assurance: 🦢

Release: 📡



Table of Contents


Quick Start

$ cat config.xml
<?xml version="1.0"?>
<!DOCTYPE module PUBLIC
          "-//Puppy Crawl//DTD Check Configuration 1.3//EN"
          "https://checkstyle.org/dtds/configuration_1_3.dtd">
<module name="Checker">
  <module name="TreeWalker">
    <module name="FallThrough"/>
  </module>
</module>

$ cat Test.java
class Test {
  public void foo() {
    int i = 0;
    while (i >= 0) {
      switch (i) {
        case 1:
        case 2:
          i++;
        case 3: // violation 'fall from previous branch of the switch'
          i++;
      }
    }
  }
}

$ java -jar checkstyle-10.18.1-all.jar -c config.xml Test.java
Starting audit...
[ERROR] Test.java:9:9: Fall through from previous branch of switch statement [FallThrough]
Audit done.
Checkstyle ends with 1 errors.

Contributing

Thanks for your interest in contributing to CheckStyle! Please see the
Contribution Guidelines
for information on how to contribute to the project. This includes creating issues, submitting pull
requests, and setting up your development environment.


Build Instructions

Please see the CheckStyle Documentation for
information on how to build the project.


Feedback and Support

  • Visit our Discussions Page, where you
    can ask questions and discuss the project with other users and contributors. This is our
    preferred method of communication for topics
    like usage and configuration questions, debugging, and other feedback.
  • Stack Overflow is another place to
    ask questions about Checkstyle usage.
  • If you are interested in contributing to the project, you can join our
    Discord Contributors Chat
    with invite link.
  • Our Google Groups Forum is a
    mailing list for discussion and support; however, we may be slow to respond there.

Javadoc

Take a look at our javadoc to see
our API documentation.


Sponsor Checkstyle

Checkstyle is an open-source project that is developed and maintained by volunteers. If you
find Checkstyle useful, please consider sponsoring the project. Your support helps us to
maintain and improve Checkstyle.


Licensing

Checkstyle is licensed under the GNU LGPL v2.1 License.

Checkstyle uses the following libraries:


Tools

Checkstyle uses the following tools:

Continuous Integration: 🏭

Quality Assurance: 🦢

Release: 📡

Development Tools: 🛠️


Development Tools Powered by

JetBrains logo.

JProfiler logo.


@Pankraz76 Pankraz76 changed the title Pull #18589: Align rewrite readme batch Pull #18610: Align rewrite readme batch #18589 Jan 11, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 11, 2026
Comment thread README.md
[jetbrains]:https://jb.gg/OpenSource
[jetbrains img]:https://resources.jetbrains.com/storage/products/company/brand/logos/jetbrains.svg

[jprofiler]:https://www.ej-technologies.com/jprofiler

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

sort this to get natural order

Comment thread README.md Outdated
--------------------------
---

*Checkstyle is a tool that ensures adherence to a code standard or a set of best practices.*

@Pankraz76 Pankraz76 Jan 11, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

impossible to see here anything in before:

Image

yes check is really serious about quallity so we should point it out and not fore someone to use a magnifying glass 🕵️‍♂️

Image

on small screen which is important to romani its one bunch:

Image

vs:

Image

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

now its very balanced and clear:

@romani

Image

Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 11, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 11, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 11, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 11, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 11, 2026
@Pankraz76

Copy link
Copy Markdown
Author

Checkstyle - Java Code Quality Tool


Checkstyle is a tool that ensures adherence to a code standard or a set of best practices.


The latest release version can be found at
GitHub releases
or at Maven repo.

Documentation is available in HTML format, see https://checkstyle.org/checks.html .

Table of Contents

Quick Start

$ cat config.xml
<?xml version="1.0"?>
<!DOCTYPE module PUBLIC
          "-//Puppy Crawl//DTD Check Configuration 1.3//EN"
          "https://checkstyle.org/dtds/configuration_1_3.dtd">
<module name="Checker">
  <module name="TreeWalker">
    <module name="FallThrough"/>
  </module>
</module>

$ cat Test.java
class Test {
  public void foo() {
    int i = 0;
    while (i >= 0) {
      switch (i) {
        case 1:
        case 2:
          i++;
        case 3: // violation 'fall from previous branch of the switch'
          i++;
      }
    }
  }
}

$ java -jar checkstyle-10.18.1-all.jar -c config.xml Test.java
Starting audit...
[ERROR] Test.java:9:9: Fall through from previous branch of switch statement [FallThrough]
Audit done.
Checkstyle ends with 1 errors.

Contributing

Thanks for your interest in contributing to CheckStyle! Please see the
Contribution Guidelines
for information on how to contribute to the project. This includes creating issues, submitting pull
requests, and setting up your development environment.

Build Instructions

Please see the CheckStyle Documentation for
information on how to build the project.

Feedback and Support

  • Visit our Discussions Page, where you
    can ask questions and discuss the project with other users and contributors. This is our
    preferred method of communication for topics
    like usage and configuration questions, debugging, and other feedback.
  • Stack Overflow is another place to
    ask questions about Checkstyle usage.
  • If you are interested in contributing to the project, you can join our
    Discord Contributors Chat
    with invite link.
  • Our Google Groups Forum is a
    mailing list for discussion and support; however, we may be slow to respond there.

Javadoc

Take a look at our javadoc to see
our API documentation.

Sponsor Checkstyle

Checkstyle is an open-source project that is developed and maintained by volunteers. If you
find Checkstyle useful, please consider sponsoring the project. Your support helps us to
maintain and improve Checkstyle.

Licensing

Checkstyle is licensed under the GNU LGPL v2.1 License.
Checkstyle uses libraries:

Development Tools Powered by

JetBrains logo.

JProfiler logo.

@romani

romani commented Jan 11, 2026

Copy link
Copy Markdown
Member

No clue what is target of update here

Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 12, 2026
Comment thread README.md
[GitHub releases](https://github.com/checkstyle/checkstyle/releases/)
or at [Maven repo](https://repo1.maven.org/maven2/com/puppycrawl/tools/checkstyle/).

Documentation is available in HTML format, see https://checkstyle.org/checks.html .

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

typo and why no real link?

Comment thread README.md
[![][cirrusci img]][cirrusci]
[![][coverage img]][coverage]
[![][snyk img]][snyk]
[![][semaphoreci img]][semaphoreci]

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

the icons are not consistent some have the name others dont. also this is not acceisabily for blind ppl.

lets have a unique was of doing including everybody not only the lucky one having the abillity to see.

Image

Comment thread README.md
[![][backers.opencollective img]][backers.opencollective]

[![][sponsors.opencollective img]][sponsors.opencollective]
---

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

dividers help too see the sections better:

Image

Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 12, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 12, 2026
@Pankraz76

Copy link
Copy Markdown
Author

No clue what is target of update here

adding rewrite and spotless to batched. we having almost all tools, so why should we not show them all?

Comment thread README.md
[![][semaphoreci img]][semaphoreci]
[![][azure img]][azure]
[![][error prone img]][error prone]
[![][pitest img]][pitest]

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

why dont we show PMD as well here? rewrite and spotless? spotbugs ?

this should be all or nothing but not some random yes and sometimes no.

Its about quality so we could improve dev onboarding with improving readme to show different tools.

Currently its just one blob impossible to simple digest.

@romani this the main goal of PR. kindly request feedback. thx.

Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 12, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 24, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 24, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 24, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 24, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 24, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 24, 2026
Comment thread README.md
maintain and improve Checkstyle.
Checkstyle is an open-source project that is developed and maintained by volunteers.

Please consider sponsoring the project, if you find Checkstyle useful.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

If you dont mind i would suggest to put first things first. @romani

Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 24, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 24, 2026
@Pankraz76

Copy link
Copy Markdown
Author

fix spelling first....

@Pankraz76 Pankraz76 closed this Jan 24, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 24, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 24, 2026
@Pankraz76

Pankraz76 commented Jan 24, 2026

Copy link
Copy Markdown
Author

testing rewrite get first issue.

#18700

@Pankraz76 Pankraz76 reopened this Jan 24, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 24, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 24, 2026
Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 24, 2026
@romani

romani commented Jan 24, 2026

Copy link
Copy Markdown
Member

we do not need changes in README.
any other unrelated to READEME changes are not expected also, or I do not understand why they are related

@romani romani closed this Jan 24, 2026
@Pankraz76

Copy link
Copy Markdown
Author

plz tell why tools are inconsittent incomplete? we shoing some random SCA tools but not rewrite and spotless why?

Pankraz76 pushed a commit to Pankraz76/checkstyle that referenced this pull request Jan 25, 2026
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.

2 participants