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

[photon-os] Adds ecosystem and prefix for Photon #107

Merged
merged 1 commit into from
Jul 3, 2023

Conversation

captn3m0
Copy link
Contributor

@captn3m0 captn3m0 commented Jan 5, 2023

As discussed in #105 earlier. I'm also using the ecosystem field in existing advisories: https://github.com/captn3m0/photon-os-advisories/blob/main/advisories/PHSA-2016-0006.json

Advisories are already published in OSV format: https://github.com/captn3m0/photon-os-advisories/tree/main/advisories (And Automated).

I'll get those enriched over time, but this could be merged in the interim.

Signed-off-by: Nemo commits@captnemo.in

docs/schema.md Outdated Show resolved Hide resolved
@captn3m0
Copy link
Contributor Author

captn3m0 commented Jan 5, 2023

Q: Any preference between photon or Photon for the ecosystem?

@oliverchang
Copy link
Contributor

Q: Any preference between photon or Photon for the ecosystem?

We typically go with the casing that's canonical for that ecosystem. Perhaps "Photon OS" is the best and unambiguous one?

@oliverchang
Copy link
Contributor

@captn3m0 friendly ping :)

@captn3m0
Copy link
Contributor Author

captn3m0 commented Feb 8, 2023

Thanks for the ping - need to fix a few more issues alongside in the advisories. Will update here soon.

@captn3m0
Copy link
Contributor Author

captn3m0 commented Jul 1, 2023

This should be good to merge now. I've made a lot of changes to the code to ensure correctness (better tracking modified) and coverage (package versions) since. The last few commits are a good indicator of the kind/frequency of updates expected.

Copy link
Contributor

@oliverchang oliverchang left a comment

Choose a reason for hiding this comment

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

thanks! Just one more comment.

docs/schema.md Outdated Show resolved Hide resolved
Signed-off-by: Nemo <commits@captnemo.in>
@oliverchang oliverchang merged commit c98462a into ossf:main Jul 3, 2023
@captn3m0 captn3m0 deleted the patch-2 branch July 3, 2023 03:58
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