-
Notifications
You must be signed in to change notification settings - Fork 105
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
Changelog #95
Changelog #95
Conversation
@daniel-beck Added change logs from 1.243 till 1.248. Please take a look. |
Go thru linked PRs and issues to get full understanding. This is summry it's unreasonable to expect to get full understanding of each or just from a sentence or two. |
I just say the layout is barely readable, I do not request more details in the changelog itself |
@oleg-nenashev Made some adjustments, please take a look. |
LGTM 👍 |
@oleg-nenashev process wise, I wait for another LGTM or its good for merging? CI is failing but thats unrelated, this PR has no code changes. |
I think it's fine to merge since the current changelog provides valuable information. If there any additional fixes, we can add changes in follow-up PRs. |
As requested by @oleg-nenashev and @daniel-beck for PR jenkinsci/jenkins#2593 (comment).