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

Set up GH Linting action for dataone-indexer #159

Open
8 tasks
artntek opened this issue Feb 3, 2025 · 0 comments
Open
8 tasks

Set up GH Linting action for dataone-indexer #159

artntek opened this issue Feb 3, 2025 · 0 comments
Labels
devtools dev tools, ci/cd etc _onboarding Good first project for onboarding

Comments

@artntek
Copy link
Collaborator

artntek commented Feb 3, 2025

User Story

As a PR reviewer, I want to be more efficient by focusing on architecture, code logic and finding potential bugs, instead of flagging issues with syntax, coding style, deprecations, typos etc. that could easily be picked up by an automatic linting tool.

Acceptance Criteria

  • Rules must be configurable
  • Feedback automatically added as PR review comments
  • Checks against the Google Java Style Guide, with the one exception that indents are 4 spaces instead of 2
  • (some of these are negotiable) Includes the level of insight provided by IDEs (e.g. IntelliJ IDEA community edition). Examples include:
    • checks for deprecated code; example:

      deprecated since version 9 and marked for removal: 'Long(java.lang.String)'
      
    • checks for code simplifications. Examples:

      * Field can be converted to a local variable
      * Private field 'spaceAllocated' is assigned but never accessed
      * Method 'hasPermission(java.lang.String, java.lang.String)' is never used
      * 'equals("")' can be replaced with 'isEmpty()'
      
    • Javadoc linting; example: Cannot resolve symbol 'docid,' (i.e. @param in javadoc does not match actual method params)

    • spell-checking in javadoc & comments; example: Typo: In word 'peroperties'

Useful info

@artntek artntek added _onboarding Good first project for onboarding devtools dev tools, ci/cd etc labels Feb 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devtools dev tools, ci/cd etc _onboarding Good first project for onboarding
Projects
None yet
Development

No branches or pull requests

1 participant