-
Notifications
You must be signed in to change notification settings - Fork 30
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
Add release notes for v1.3.1 and v1.2.2. #17
Conversation
These releases correspond to the recently-landed security fixes. Also breaks up some long lines to make it easier to edit the markdown in a text editor.
/cc @boingoing @liminzhu |
|
||
This patch release of ChakraCore 1.3 includes the following security fixes: | ||
|
||
- [PR #1979](https://github.com/Microsoft/ChakraCore/pull/1979) |
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.
Can we use this format for release note bullets for consistency? Basically appending PR link at the end and leaving commit link if a PR link is available. It's duplicate information.
- Change to address CVE-2016-7207 #1979
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 totally agree. I'll update.
👍 |
Let's merge it! |
Will do, once the releases are actually out. Need to merge the version updates and build before we can publish the packages. |
These releases correspond to the recently-landed security fixes.
Also breaks up some long lines to make it easier to edit the markdown in a text editor.
Ref chakra-core/ChakraCore#1979
Ref chakra-core/ChakraCore#1982