Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jenkins Checkstyles: Explore the Viability of GitHub Integration #7346

Open
2 tasks
JackEAllen opened this issue Sep 9, 2022 · 0 comments
Open
2 tasks

Jenkins Checkstyles: Explore the Viability of GitHub Integration #7346

JackEAllen opened this issue Sep 9, 2022 · 0 comments
Labels
Friday no_release_notes Tickets that do not need release notes, use sparingly!

Comments

@JackEAllen
Copy link
Member

As a: developer I would like Checkstyles to stop increasing and begin reducing in size through means of a stricter review process by integrating existing checkstyle static code analysis performed by Jenkins into GitHub as an on push action.

By doing this, checks will be displayed in GitHub for each branch and easier to include as part of a review when a PR has been created to merge into production. It is then up to the reviewers discretion to let branches be merged in if time critical if checkstyles increase, and to otherwise ask for a rework to resolve, passing only if the checkstyles remain the same or decrease in quantity.

In response to a discussion taken place in the retrospective Retrospective notes 2022.09.07, the purpose of this issue is to explore the viability of making this possible and creating an issue to make this possible if easily viable.

Acceptance Criteria

What is the acceptance criteria?

  • Viability of integrating checkstyle code analysis through Jenkins into GitHub on push has been explored and comments on research added to this issue.
  • A new detailed issue exists to integrate into GitHub on push for branches if viable to display if checkstyles has increased or decreased in quantity.

Extra Information

A resource which may be of use when looking into the viability which is an existing GitHub action to do just this as a GitHub workflow rather than through Jenkins can be seen below:

How to Test

verbose instructions for reviewer to test changes
(Add before making a PR)

@JackEAllen JackEAllen added Friday no_release_notes Tickets that do not need release notes, use sparingly! labels Sep 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Friday no_release_notes Tickets that do not need release notes, use sparingly!
Projects
None yet
Development

No branches or pull requests

1 participant