-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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 5.8.1 #7957
CHANGELOG 5.8.1 #7957
Conversation
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.
Some minor improvements requested.
### Documentation | ||
|
||
- Add note about windows_security_products compatibility ([#7880](https://github.com/osquery/osquery/pull/7880)) | ||
- CHANGELOG 5.7.0 ([#7894](https://github.com/osquery/osquery/pull/7894)) |
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.
- CHANGELOG 5.7.0 ([#7894](https://github.com/osquery/osquery/pull/7894)) |
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.
Eh? Why remove this?
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 don't see any reason why a user would benefit from seeing that the changelog was updated for 5.7 in the 5.8.1 release notes. I don't see much harm either though so if you want to leave it there that's fine with me.
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 agree it's of low value, but I believe in being complete about them. I've generally tended to including all changes.
Thank you for the suggestions. I think we probably need better hygiene on PR titles Co-authored-by: Zach Wasserman <[email protected]>
No description provided.