Skip to content

Remove checkstyle hack. 8.36 got released#6816

Merged
Siedlerchr merged 2 commits into
masterfrom
update-checkstyle-to-8.36
Aug 31, 2020
Merged

Remove checkstyle hack. 8.36 got released#6816
Siedlerchr merged 2 commits into
masterfrom
update-checkstyle-to-8.36

Conversation

@koppor

@koppor koppor commented Aug 31, 2020

Copy link
Copy Markdown
Member

Checkstyle 8.36 was released - including Java14 support: https://checkstyle.org/releasenotes.html. Thus, no hack necessary any more.

  • Change in CHANGELOG.md described (if applicable)
  • Tests created for changes (if applicable)
  • Manually tested changed features in running JabRef (always required)
  • Screenshots added in PR description (for UI changes)
  • Checked documentation: Is the information available and up to date? If not created an issue at https://github.com/JabRef/user-documentation/issues or, even better, submitted a pull request to the documentation repository.

@Siedlerchr

Copy link
Copy Markdown
Member

Seems like we need to reformat the previously excluded classes (import order and space comment)

@koppor

koppor commented Aug 31, 2020

Copy link
Copy Markdown
Member Author

Checkstyle fixed in src/main/java/org/jabref/gui/EntryTypeView.java

@koppor koppor added the status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers label Aug 31, 2020
@Siedlerchr Siedlerchr merged commit 524b965 into master Aug 31, 2020
@Siedlerchr Siedlerchr deleted the update-checkstyle-to-8.36 branch August 31, 2020 13:08
Siedlerchr added a commit that referenced this pull request Aug 31, 2020
* upstream/master: (120 commits)
  Follow up fix for copy paste (#6820)
  Add CSS Customisation (#6725)
  Separate signing and notarizing (#6822)
  Remove checkstyle hack. 8.36 got released (#6816)
  Feature/enable lucene query parsing (#6799)
  New release cycle
  Bump WyriHaximus/github-action-wait-for-status from v1.1.2 to v1.2 (#6814)
  Bump mockito-core from 3.5.5 to 3.5.7 (#6813)
  Bump classgraph from 4.8.87 to 4.8.89 (#6812)
  Bump me.champeau.gradle.jmh from 0.5.0 to 0.5.1 (#6811)
  Bump checkstyle from 8.35 to 8.36 (#6810)
  Improve Changelog
  Refactor edit action (#6808)
  Fixed typo in BuildInfo (#6807)
  disable checkstyle for generated
  fix checkstyle
  Simplify check-links.yaml (markdown-link-check) (#6720)
  Rename /gen to /generate (#6800)
  Disable CSL refresh on push (#6803)
  New Crowdin updates (#6804)
  ...

# Conflicts:
#	src/main/java/org/jabref/gui/JabRefFrame.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants