8253206: Enforce whitespace checking for additional source files #195
Conversation
/label add build |
|
@kevinrushforth |
@kevinrushforth |
@kevinrushforth To determine the appropriate audience for reviewing this pull request, one or more labels corresponding to different subsystems will normally be applied automatically. However, no automatic labelling rule matches the changes in this pull request. In order to have an RFR email automatically sent to the correct mailing list, you will need to add one or more labels manually using the |
/reviewers 2 |
@kevinrushforth Only Reviewers are allowed to change the number of required reviewers. |
@kevinrushforth This change now passes all automated pre-integration checks. In addition to the automated checks, the change must also fulfill all project specific requirements After integration, the commit message will be:
Since the source branch of this PR was last updated there have been 14 commits pushed to the
As there are no conflicts, your changes will automatically be rebased on top of these commits when integrating. If you prefer to avoid automatic rebasing, please merge As you do not have Committer status in this projectan existing Committer must agree to sponsor your change. Possible candidates are the reviewers of this PR (@prrace, @erikj79, @jayathirthrao) but any other Committer may sponsor as well.
|
/integrate |
@kevinrushforth |
/sponsor |
@jayathirthrao @kevinrushforth Since your change was applied there have been 14 commits pushed to the
Your commit was automatically rebased without conflicts. Pushed as commit 1086713. |
This adds the following extensions to the list of source files that git jcheck will check for whitespace errors:
All files with the above extensions are now white-space clean after the fix for JDK-8240487. This will help them stay that way.
I just integrated a similar fix to
jfx
(with a few more source extensions that aren't relevant for the JDK) in JDK-8240499.Progress
Issue
Reviewers
Download
$ git fetch https://git.openjdk.java.net/jdk pull/195/head:pull/195
$ git checkout pull/195