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

Add intro to ArmorCode integration #1205

Merged
merged 2 commits into from
Oct 19, 2023
Merged

Conversation

martijnrusschen
Copy link
Member

No description provided.

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.


Check the status or cancel PullRequest code review here.

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

+ 5
- 0

100% Markdown

Type of change

Minor Update - These changes appear to be a minor update to existing functionality and features.
1 Message
📚 It looks like the description for this pull request is either blank or very short. Adding a high-level summary will help our reviewers provide better feedback. Feel free to include questions for PullRequest reviewers and make specific feedback requests.

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 have just one minor note left inline.

Image of Graham C Graham C


Reviewed with ❤️ by PullRequest


ArmorCode unifies the siloed world of security testing so you can move faster and spend time on what matters most. The ArmorCode-HackerOne integration makes it easy to ingest and correlate findings from HackerOne to the ArmorCode platform, prioritize based on risk, and automate triaging and remediation workflows to improve your security posture at scale.

To learn more, visit the [ArmorCode website](https://www.armorcode.com).
Copy link

Choose a reason for hiding this comment

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

Strictly speaking, we need a trailing / on the URL here (even if some browsers silently hide it from the URL bar - it is actually still there).

https://www.armorcode.com/

🔸 Bug (Important)

Image of Graham C Graham C

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks, good catch

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 reviewed the updates made to #1205 up until the latest commit (d4e985a). No further issues were found.

Reviewed by:

Image of Graham C Graham C

@martijnrusschen martijnrusschen merged commit 9a6c176 into master Oct 19, 2023
3 checks passed
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.

1 participant