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 user list for matrix #81

Merged
merged 2 commits into from
Apr 17, 2024
Merged

Update user list for matrix #81

merged 2 commits into from
Apr 17, 2024

Conversation

jasonbrooks
Copy link
Contributor

Updating this list of handles so the pings work on matrix. If I saw a matrix.org handle and a fedora.im one, I chose the fedora.im one. I couldn't find a few folks: (anthr76 jcajka nasirhm saqali) so I removed them.

Updating this list of handles so the pings work on matrix. If I saw a matrix.org handle and a fedora.im one, I chose the fedora.im one. I couldn't find a few folks: (anthr76 jcajka nasirhm saqali) so I removed them.
@dustymabe
Copy link
Member

dustymabe commented Apr 17, 2024

Thanks @jasonbrooks - I think two things need to happen. This isn't your fault.

  1. we should update this script to grab the content from https://github.com/coreos/fedora-coreos-tracker/blob/main/meeting-people.txt - this is The meeting template should pull the people list from the tracker repo #77
  2. we should update the meeting people there and not here

Though this can merge as is IMO since it is an improvement.

@dustymabe
Copy link
Member

@prestist - want to merge this as is or take on the bigger change mentioned above ?

@jasonbrooks
Copy link
Contributor Author

Thanks @jasonbrooks - I think two things need to happen. This isn't your fault.

  1. we should update this script to grab the content from https://github.com/coreos/fedora-coreos-tracker/blob/main/meeting-people.txt
  2. we should update the meeting people there and not here

I opened a PR over there too, and added here some names that were in the other location

Though this can merge as is IMO since it is an improvement.

@prestist
Copy link
Contributor

Yeah, I dont see an issue with merging this now. I do agree @dustymabe we certainly can add that. I dont think it will be hard, and I can see about doing so soon.

@prestist prestist merged commit 8c6bc88 into coreos:main Apr 17, 2024
1 check passed
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.

None yet

3 participants