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

Add roles yaml section to Vulnerability Management WG #746

Merged
merged 2 commits into from
Jan 11, 2024

Conversation

beyhan
Copy link
Member

@beyhan beyhan commented Jan 10, 2024

This is need for the inactive user automation to not suggest users for deletion who has roles. cc @paulcwarren

@beyhan beyhan force-pushed the add-vulnerability-management-wg-roles branch from 22d9cd9 to 9eb4b97 Compare January 10, 2024 09:09
github: ijzerman
- name: Khushboo Soni
github: skhushboo-vm
- name: Mathias Essenpreis
Copy link
Member

Choose a reason for hiding this comment

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

No longer active

Copy link
Member Author

Choose a reason for hiding this comment

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

Done

github: skhushboo-vm
- name: Mathias Essenpreis
github: Essenpreis
- name: Prateek Mehrotra
Copy link
Member

Choose a reason for hiding this comment

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

No longer active

Copy link
Member Author

Choose a reason for hiding this comment

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

Done

Copy link
Member

@paulcwarren paulcwarren left a comment

Choose a reason for hiding this comment

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

Can we remove the inactive users please?

@beyhan beyhan force-pushed the add-vulnerability-management-wg-roles branch from 3b9f559 to e071754 Compare January 11, 2024 07:23
@beyhan
Copy link
Member Author

beyhan commented Jan 11, 2024

Can we remove the inactive users please?

@paulcwarren is it OK now?

@beyhan beyhan requested a review from paulcwarren January 11, 2024 07:24
@beyhan
Copy link
Member Author

beyhan commented Jan 11, 2024

I'm going to merge this because org automation wasn't in place and no roles were given to the removed people.

@beyhan beyhan requested review from a team, stephanme, ameowlia and ChrisMcGowan and removed request for a team January 11, 2024 16:07
@beyhan beyhan merged commit f97f5f3 into main Jan 11, 2024
2 checks passed
@beyhan beyhan deleted the add-vulnerability-management-wg-roles branch January 11, 2024 16:30
beyhan added a commit that referenced this pull request Jan 12, 2024
With [pr](#746) we
introduced roles yaml for the VM WG. We missed to add the members
as contributors with that pr.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants