Skip to content
This repository has been archived by the owner on Nov 20, 2023. It is now read-only.

CVE update #1217

Merged
merged 2 commits into from
Oct 26, 2023
Merged

CVE update #1217

merged 2 commits into from
Oct 26, 2023

Conversation

jessiwright
Copy link
Contributor

per: https://gitlab.inverselink.com/hackerone/issues/-/issues/68010
minor updates to existing document, new section added to bottom

Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ This pull request was sent to the PullRequest network.


@jessiwright you can click here to see the review status or cancel the code review job.

Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PullRequest Breakdown

Reviewable lines of change

+ 20
- 9

100% Markdown
<1% Other

Type of change

Minor Update - These changes appear to be a minor update to existing functionality and features.

Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work here. I don't have any concerns. 🎉

Image of Graham C Graham C


Reviewed with ❤️ by PullRequest

@jessiwright jessiwright requested review from martijnrusschen and removed request for martijnrusschen October 25, 2023 18:46
Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Numbering everything with 1 is a good practice, because it allows you to insert items without changing all the numbers. They get rendered as 1, 2, 3, 4, etc. I'd recommend reverting the last change.

Image of Graham C Graham C


Reviewed with ❤️ by PullRequest

@jessiwright jessiwright merged commit f87c2e9 into master Oct 26, 2023
5 checks passed
@jessiwright jessiwright deleted the cve-update branch October 26, 2023 13:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants