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

Ccullen cert patch 2 #512

Merged
merged 10 commits into from
Feb 28, 2024
Merged

Conversation

ccullen-cert
Copy link
Contributor

@ccullen-cert ccullen-cert commented Feb 28, 2024

This request is to address #493 and #381.
Its showing I modified index.md but the change is the same.

Copy link
Contributor

@ahouseholder ahouseholder left a comment

Choose a reason for hiding this comment

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

I'm approving this since the net change is ok.

The change history on this branch is a little weird because it looks like it started as a branch off of your own fork's main which was already out of sync with CERTCC/SSVC:main, but that's going to disappear as soon as we squash merge this. Not a big deal for this one, but starting future branches from CERTCC/SSVC:main might make it easier to work with.

@ahouseholder ahouseholder merged commit 2590cad into CERTCC:main Feb 28, 2024
2 checks passed
@ahouseholder ahouseholder added this to the SSVC 202403 milestone Feb 28, 2024
@ahouseholder ahouseholder added bug Something isn't working documentation Improvements or additions to documentation labels Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
None yet
2 participants