-
Notifications
You must be signed in to change notification settings - Fork 39
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
📖 Added a walkthrough of how to create a custom rule #681
base: main
Are you sure you want to change the base?
📖 Added a walkthrough of how to create a custom rule #681
Conversation
docs/custom_ruleset.md
Outdated
> The Maven artifact io.fabric8:kubernetes-model has been removed from the project and is no longer published. | ||
> This artifact was just an aggregator of some of the Kubernetes model artifacts and had no specific purpose. It is no longer published, the io.fabric8:kubernetes-client-api or io.fabric8:kubernetes-openshift-uberjar artifacts should be used instead. | ||
|
||
So for this, we will need to make sure that if the dependency `io.fabric8:kubernetes-model` is not used. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Worth a 2nd read of this sentence, maybe a typo, maybe wanted to remove the word "if"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated let me know if the new wording makes more sense.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, just saw 1 line you may want to tweak.
Signed-off-by: Shawn Hurley <[email protected]>
1d4c565
to
a766b25
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am the king of nits
Co-authored-by: Jonah Sussman <[email protected]> Signed-off-by: Shawn Hurley <[email protected]>
Co-authored-by: Jonah Sussman <[email protected]> Signed-off-by: Shawn Hurley <[email protected]>
Co-authored-by: Jonah Sussman <[email protected]> Signed-off-by: Shawn Hurley <[email protected]>
No description provided.