Skip to content

implement the removal of members - #376

Merged
garloff merged 3 commits into
mainfrom
feat/remove-members
Sep 5, 2026
Merged

implement the removal of members#376
garloff merged 3 commits into
mainfrom
feat/remove-members

Conversation

@fkr

@fkr fkr commented Jun 5, 2026

Copy link
Copy Markdown
Member

Assisted-by: claude-haiku-4-5, claude-sonnet-4-6

Assisted-by: claude-haiku-4-5, claude-sonnet-4-6
Signed-off-by: Felix Kronlage-Dammers <fkr@hazardous.org>
@fkr
fkr requested a review from a team as a code owner June 5, 2026 15:45
@fkr

fkr commented Jun 5, 2026

Copy link
Copy Markdown
Member Author

Functionality has been tested first with --dry-run and then by actually making sure that our org and the data.yaml have the same state.

run: |
pipenv run python3 remove_members.py --dry-run
env:
API_TOKEN: ${{ secrets[format('GHP_{0}', github.actor)] }}
run: |
pipenv run python3 remove_members.py
env:
API_TOKEN: ${{ secrets[format('GHP_{0}', github.actor)] }}

@garloff garloff left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, thanks!

@garloff

garloff commented Sep 3, 2026

Copy link
Copy Markdown
Member

Now, we only need to find out why the GH action fails.

@jklare

jklare commented Sep 4, 2026

Copy link
Copy Markdown

Now, we only need to find out why the GH action fails.

I am not sure if this is related, but we also had quite some trouble with GitHub and our pipelines lately related to this: https://github.com/orgs/community/discussions/206581

@garloff

garloff commented Sep 5, 2026

Copy link
Copy Markdown
Member

It was just my expired PAT ...

@garloff
garloff merged commit 64250db into main Sep 5, 2026
7 checks passed
@garloff
garloff deleted the feat/remove-members branch September 5, 2026 07:54
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.

4 participants