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.

Sign up for GitHub

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

Jump to bottom

nonspec: Add Marcela as new SLSA spec maintainer #1114

Merged
merged 1 commit into from
Aug 20, 2024
Merged

nonspec: Add Marcela as new SLSA spec maintainer #1114

merged 1 commit into from
Aug 20, 2024

Conversation

marcelamelara
Copy link
Contributor

I'd like to self-nominate for maintainer status of the SLSA spec. I believe I meet the criteria for maintainer status. Thanks!

@netlify Netlify
Copy link

netlify bot commented Aug 15, 2024 •
edited
Loading

✅ Deploy Preview for slsa canceled.

Name Link
🔨 Latest commit 7ffa4b0
🔍 Latest deploy log https://app.netlify.com/sites/slsa/deploys/66be762f40b8920008996101

@marcelamelara marcelamelara changed the title Add Marcela as new SLSA spec maintainer nospec: Add Marcela as new SLSA spec maintainer Aug 15, 2024
@marcelamelara marcelamelara changed the title nospec: Add Marcela as new SLSA spec maintainer nonspec: Add Marcela as new SLSA spec maintainer Aug 15, 2024
lehors
lehors approved these changes Aug 16, 2024
View reviewed changes
Copy link
Member

@lehors lehors 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 all for it! We can certainly use the additional help given how several of us have been pulled into other things and can't dedicate as much time as we used to this project.

TomHennen
TomHennen approved these changes Aug 16, 2024
View reviewed changes
Copy link
Contributor

@TomHennen TomHennen left a comment

Choose a reason for hiding this comment

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

I fully support this! Thanks for the proposal. :)

arewm
arewm approved these changes Aug 16, 2024
View reviewed changes
Copy link
Member

@arewm arewm left a comment

Choose a reason for hiding this comment

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

Thanks for all that you have been doing!

joshuagl
joshuagl approved these changes Aug 20, 2024
View reviewed changes
Copy link
Member

@joshuagl joshuagl left a comment

Choose a reason for hiding this comment

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

Thanks for self nominating. I wholeheartedly support this.

@TomHennen
Copy link
Contributor

I think we're good to go. I'll merge this and add Marcela to the appropriate group.

@TomHennen TomHennen merged commit ac723d1 into slsa-framework:main Aug 20, 2024
6 of 8 checks passed
@marcelamelara
Copy link
Contributor Author

Thank you all :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TomHennen TomHennen TomHennen approved these changes

@lehors lehors lehors approved these changes

@joshuagl joshuagl joshuagl approved these changes

@arewm arewm arewm approved these changes

Assignees
No one assigned
Labels
None yet
Projects
Issue triage
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@marcelamelara @TomHennen @lehors @joshuagl @arewm