-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
commons-digester:2.1 removal #4395
Conversation
First shot, refining
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 so far, no typos I can tell
thanks @zbynek! Co-authored-by: Zbynek Konecny <zbynek1729@gmail.com>
Thanks @jtnord
Thanks @jlamasrios !
I think I have addressed all the feedback. This is ready for review and possibly merge to me @jenkins-infra/copy-editors Thanks all. |
|
||
=== Resources | ||
|
||
link:https://github.com/jenkinsci/jenkins/pull/5320[PR-5320]:: removing `commons-digester:2.1`. |
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.
a screenshot of the post in the description would be good to include to make sure all this renders correctly
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.
Care to elaborate? I'm confused by this request.
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.
normally someone would take a full page screenshot of a post and put it in the description so others don't have to check it out locally.
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.
oh ok, I generally don't do that for simple entries with text, sticking with GH renderer. Looking into this then.
This blog entry provides a heads-up on the Core PR that has been merged this morning, we should publish this soon enough.
jenkinsci/jenkins#5320