Skip to content

Buzzsurfr update #663

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

Closed
wants to merge 2 commits into from
Closed

Buzzsurfr update #663

wants to merge 2 commits into from

Conversation

buzzsurfr
Copy link

No description provided.

lukaszgryglicki and others added 2 commits April 16, 2025 16:04
Signed-off-by: Lukasz Gryglicki <[email protected]>
Signed-off-by: Theo Salvo <[email protected]>
Copy link
Member

@lukaszgryglicki lukaszgryglicki left a comment

Choose a reason for hiding this comment

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

Please follow instructions in: https://github.com/cncf/gitdm?tab=readme-ov-file#addingupdating-affiliation, especially:

Only the Developers affiliations list [dev1](https://github.com/cncf/gitdm/blob/master/developers_affiliations1.txt), [dev2](https://github.com/cncf/gitdm/blob/master/developers_affiliations2.txt), [dev3](https://github.com/cncf/gitdm/blob/master/developers_affiliations3.txt), [dev4](https://github.com/cncf/gitdm/blob/master/developers_affiliations4.txt), [dev5](https://github.com/cncf/gitdm/blob/master/developers_affiliations5.txt), ... should be edited manually.

Company Developers lists [co1](https://github.com/cncf/gitdm/blob/master/company_developers1.txt), [co2](https://github.com/cncf/gitdm/blob/master/company_developers2.txt), [co3](https://github.com/cncf/gitdm/blob/master/company_developers3.txt), [co4](https://github.com/cncf/gitdm/blob/master/company_developers4.txt), [co5](https://github.com/cncf/gitdm/blob/master/company_developers5.txt), [co6](https://github.com/cncf/gitdm/blob/master/company_developers6.txt), [co7](https://github.com/cncf/gitdm/blob/master/company_developers7.txt), [co8](https://github.com/cncf/gitdm/blob/master/company_developers8.txt) are computed derivatives of the first list.

I also saw that many changes are moving the same affiliations data from one file to another resulting in a huge PR modifying 23 files. Please update that, so I can review again

@lukaszgryglicki
Copy link
Member

lukaszgryglicki commented Apr 18, 2025

So please edit only Developers affiliations files - adding new entries or updating existing ones. You can add all new entries in the last file - not bother with sorting, etc. - the import tool will do that after merge.

@lukaszgryglicki
Copy link
Member

There are conflicts and sections of files moved to another files, please update in a way I've asked, eventually open a fresh new PR. Thanks.

@lukaszgryglicki
Copy link
Member

This also attempts to update github_users.json which is LFS - closing this - please fix/update and reopen or create a new PR.

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.

2 participants