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

feat: forced prefixes #495

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

l3r8yJ
Copy link
Contributor

@l3r8yJ l3r8yJ commented Feb 20, 2025

@volodya-lombrozo take a look, please

I've developed a plugin, that checks whether is prefix was set or not


PR-Codex overview

This PR focuses on renaming classes and annotations to improve clarity and consistency in the codebase. It also introduces a new dependency and plugin related to prefix requirements.

Detailed summary

  • Renamed LineHitterRule to RuleLineHitter for consistency.
  • Renamed Text class to ComplaintText in Complaint.
  • Updated annotation @SuppressWarnings("JTCOP.LineHitterRule") to @SuppressWarnings("JTCOP.RuleLineHitter").
  • Added @RequirePrefix annotations to Rule and Complaint interfaces.
  • Introduced new Maven dependency for prefixed-maven-plugin.
  • Added a new plugin configuration for prefixed-maven-plugin.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@l3r8yJ
Copy link
Contributor Author

l3r8yJ commented Feb 20, 2025

@volodya-lombrozo seems like it's problem with 11 version of Java, i'll comeback a bit later

@l3r8yJ
Copy link
Contributor Author

l3r8yJ commented Feb 20, 2025

@volodya-lombrozo can you restar pipe a bit later, dependency not updated in maven central

@l3r8yJ
Copy link
Contributor Author

l3r8yJ commented Feb 21, 2025

@volodya-lombrozo up

@volodya-lombrozo
Copy link
Owner

@l3r8yJ It seems that the plugin is incompatible with Java 11. Am I right?

@l3r8yJ
Copy link
Contributor Author

l3r8yJ commented Feb 24, 2025

@volodya-lombrozo I've fixed it, now we need just to restart CI and everything should be okay

update: not okay, I'll fix it

@l3r8yJ
Copy link
Contributor Author

l3r8yJ commented Feb 24, 2025

@volodya-lombrozo you can restart the actions to re-pull the dependency

@volodya-lombrozo
Copy link
Owner

@l3r8yJ There are still some issues

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