Skip to content

Conversation

@eddymul
Copy link
Contributor

@eddymul eddymul commented Jun 15, 2023

No description provided.

@eddymul eddymul self-assigned this Jun 15, 2023
@eddymul eddymul marked this pull request as ready for review June 16, 2023 00:10
@eddymul eddymul requested review from Andrew5194 and ethan42 June 16, 2023 00:10
| `runId` | string | The identifier of the run that this action triggered in Mayhem. |


📖 See the [CI/CD](https://mayhem.forallsecure.com/docs/mayhem-ci-cd/fuzzing-in-your-pipeline/) docs for more information and guides on using the mCode GitHub Action!
Copy link
Member

Choose a reason for hiding this comment

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

at the end of the readme there are also some references to forallsecure.com. I would recommend changing them to mayhem.security.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should be resolved in e889175

But, would appreciate another look. 🙏

@ethan42 ethan42 requested a review from vlussenburg June 16, 2023 08:32
Copy link
Contributor

@Andrew5194 Andrew5194 left a comment

Choose a reason for hiding this comment

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

LGTM, left a few suggestions

Co-authored-by: Andrew5194 <20665775+Andrew5194@users.noreply.github.com>
@eddymul eddymul merged commit fdeb75b into main Jun 17, 2023
@mjkoo mjkoo deleted the app-mayhem-security branch July 12, 2023 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants