-
Notifications
You must be signed in to change notification settings - Fork 219
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
Syncing CVE Pages Across Docs/i18n Versions #1589
Conversation
Signed-off-by: Sunil Singh <[email protected]>
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.
The starting pipe character is missing from the start of each row.
Note: also seems to be the case for original PR that's referenced.
Signed-off-by: Sunil Singh <[email protected]>
@btat Thanks for catching Billy! I've pushed a fix to address the missing pipe character between versions, and also realized that the i18n CVE versions were out of sync for a few updates so added those in with this PR as well. Let me know if anything needs to be adjusted, thank you! |
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.
Thanks @sunilarjun LGTM. One small nit is that 22030 could have its translation preserved, but adding an approval since it's not a blocker.
Signed-off-by: Sunil Singh <[email protected]>
Signed-off-by: Sunil Singh <[email protected]>
@btat Thanks for catching that! Added the translated version back in and added back an incorrectly removed CVE from v2.7, please let me know if anything else needs editing. |
Syncing v2.10 CVE page as some CVE's were missing from previous updates.