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

Update operations.yml to remove inactive EWG members #2551

Closed
wants to merge 4 commits into from

Conversation

nataled
Copy link
Contributor

@nataled nataled commented Mar 19, 2024

No description provided.

@@ -46,7 +46,6 @@ members:
country: USA
github: wdduncan
groups:
- editorial
Copy link
Contributor

Choose a reason for hiding this comment

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

What does it mean to have someone on the Ops committee with no assigned groups? Don't all active Ops members have to belong to one or more groups?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That's my understanding but my memory is fuzzy on that. Heck, there are many members listed that never join meetings. Anyway, this is to address the EWG issue only. Anything beyond that is out of my scope and will need to be brought up at an Ops meeting.

Copy link
Contributor

Choose a reason for hiding this comment

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

I'll add it to the agenda

Copy link
Contributor

Choose a reason for hiding this comment

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

This is why the PR is failing. If a person doesn't have a group assigned to him or her, you have to remove the groups: key, too.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you @anitacaron !

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@anitacaron Either I did something wrong, or there is something else causing the failure. I fixed the issue you mentioned but things are still failing. Perhaps I should just start over.

@anitacaron anitacaron linked an issue Mar 20, 2024 that may be closed by this pull request
- editorial
link: https://www.lji.org/labs/peters/
- outreach
link: https://www.lji.org/labs/peters/
Copy link
Contributor

Choose a reason for hiding this comment

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

@nataled Here's the issue: when changed the group, you removed a space before link. 😄

Copy link
Contributor Author

Choose a reason for hiding this comment

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

-_- so these files are sensitive to spaces except when I want them to be. Got it! 😄

Anyway, I just re-did the whole thing from the start, and will delete this branch when I figure out how.

@nataled nataled closed this Mar 20, 2024
@nataled nataled deleted the nataled-patch-45 branch March 20, 2024 15:52
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.

Update the list of members of the Editorial Working Group
3 participants