Skip to content

Add HERE Proprietary rule for pom.xml files - #3212

Merged
pombredanne merged 2 commits into
aboutcode-org:developfrom
bennati:here-rule
Jan 21, 2023
Merged

Add HERE Proprietary rule for pom.xml files#3212
pombredanne merged 2 commits into
aboutcode-org:developfrom
bennati:here-rule

Conversation

@bennati

@bennati bennati commented Jan 20, 2023

Copy link
Copy Markdown
Contributor

Signed-off-by: Bennati, Stefano stefano.bennati@here.com

Tasks

  • Reviewed contribution guidelines
  • PR is descriptively titled 📑 and links the original issue above 🔗
  • Tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR
    Run tests locally to check for errors.
  • Commits are in uniquely-named feature branch and has no merge conflicts 📁

Signed-off-by: Bennati, Stefano <stefano.bennati@here.com>

@pombredanne pombredanne left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! Thanks!

This avoids any ambiguity with other rules.

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>

@pombredanne pombredanne left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

See this refinement b8d0b57 that avoids any ambiguity with other rules. (We need to improve these POM-related rules alright anyway).

@pombredanne

Copy link
Copy Markdown
Member

Thanks! merging

@pombredanne
pombredanne merged commit cad392f into aboutcode-org:develop Jan 21, 2023
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