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

Extend inactive user automation to warn about inactive WG users #918

Merged
merged 2 commits into from
Jul 9, 2024

Conversation

beyhan
Copy link
Member

@beyhan beyhan commented Jul 3, 2024

As discussed during the TOC meeting on 02.07.2024 this change extends the inactive user automation to warn about WG users who were inactive during the last one year. If this changes is accepted the inactive pr msg will look like:

According to the rules for inactivity defined in RFC-0025 following users will be deleted:
user1
user2
....
According to the revocation policy in the RFC, users have two weeks to refute this revocation, if they wish by commenting on this pull-request and open a new pull-request to be re-added as contributor after this one is merged.
As alternative, if you are active in a working group please check the promotion rules and if you are eligible and wish apply for a role in that working group.

Warning:
Inactive users of Working Group "CF on K8s" are:
user3
...
Inactive users of Working Group "Service Management" are:
user4
...
Inactive users of Working Group "App Runtime Interfaces" are:
user5
...
Inactive users of Working Group "App Runtime Platform" are:
...

@beyhan beyhan requested review from a team, rkoster, stephanme, ameowlia and ChrisMcGowan and removed request for a team July 3, 2024 14:32
@beyhan beyhan added the toc label Jul 3, 2024
@beyhan beyhan force-pushed the extend-inactive-user-automation-for-wg-users branch from c8953e1 to 2848049 Compare July 5, 2024 11:55
@beyhan beyhan force-pushed the extend-inactive-user-automation-for-wg-users branch from 2848049 to d380858 Compare July 5, 2024 12:03
Copy link
Contributor

@stephanme stephanme left a comment

Choose a reason for hiding this comment

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

There are code style errors unrelated to this change, probably due to a newer BLACK version:

(cf-community-3.9) D047883@VR9NDXJ7JP org % python -m flake8
./org_management.py:447:56: BLK100 Black would make changes.

Can be fixed separately. Need to think about how to integrate python linting and testing into PR validation if org/*.py files are touched.

@ameowlia ameowlia merged commit 8965767 into main Jul 9, 2024
2 checks passed
@ameowlia ameowlia deleted the extend-inactive-user-automation-for-wg-users branch July 9, 2024 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

6 participants