Skip to content
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

Updated report examples and replaced the full example with an image in the README #206

Merged
merged 4 commits into from
Jan 25, 2024

Conversation

anna-aleksandrowicz
Copy link
Contributor

Updated report examples and replaced the full example with an image

Updated report examples and replaced the full example with an image
MaorDavidzon
MaorDavidzon previously approved these changes Jan 25, 2024
@MarshalX MarshalX changed the title Update README.md Updated report examples and replaced the full example with an image in the README Jan 25, 2024
@MarshalX
Copy link
Collaborator

MarshalX commented Jan 25, 2024

@anna-aleksandrowicz could you pls reupload you image to https://github.com/cycodehq/cycode-cli/tree/main/images? as we have the others

@MarshalX
Copy link
Collaborator

lemme provide another img because this one doesnt contain "report url" for docs

@MarshalX
Copy link
Collaborator

MarshalX commented Jan 25, 2024

sca_report_url

@MarshalX
Copy link
Collaborator

MarshalX commented Jan 25, 2024

@MaorDavidzon could we merge it without waiting for @anna-aleksandrowicz approve? @saramontif needs it for new release ASAP

@MaorDavidzon
Copy link
Contributor

@MaorDavidzon could we merge it without waiting for @anna-aleksandrowicz approve? @saramontif needs it for new release ASAP

Yes it's fine
Let's update the documentation early next week

@MarshalX MarshalX merged commit 9465e88 into cycodehq:main Jan 25, 2024
25 checks passed
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.

3 participants