Skip to content

Update New Project Playbook#61

Merged
cunla merged 10 commits intomainfrom
new-proj-changes
Nov 14, 2024
Merged

Update New Project Playbook#61
cunla merged 10 commits intomainfrom
new-proj-changes

Conversation

@cunla
Copy link
Member

@cunla cunla commented Oct 22, 2024

No description provided.

@cunla cunla requested a review from a team October 22, 2024 18:37
@cunla cunla requested review from a team and tim-schilling November 1, 2024 13:01
@ryancheley
Copy link
Member

@cunla and @tim-schilling these look like great changes! It makes the process a little more clear and does a great job of calling out who needs to do what, and in what order 💯

### Pre Transfer Steps

- [ ] Check if the repository meets [inbound requirements][3].
- [ ] Confirm who will be the admins and maintainers for the repository
Copy link
Member

Choose a reason for hiding this comment

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

Extra thoughts from my notes

  • Consider removing any admins that are no longer active
  • Make sure the current contributors with some form of write access are members of Django Commons. Alternatively, they can apply for membership later and be added to the repository again.

Copy link
Member Author

Choose a reason for hiding this comment

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

Removing admins/collaborators manually no longer needed, it will be done as part of the terraform import process - see this PR: django-commons/membership#98

### Make GitHub repository managed by terraform

- [ ] Terraform changes to add project to organization
- [ ] Terraform changes to add project to organization, should be included in the issue opened to transfer the project.
Copy link
Member

Choose a reason for hiding this comment

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

I have written in my notes to discuss our use of Terraform. For example, most changes normally done through the GitHub UI should now be done using Terraform. Adding new members for example.

@cunla cunla requested review from a team and Stormheg November 2, 2024 14:48
Copy link
Member

@tim-schilling tim-schilling left a comment

Choose a reason for hiding this comment

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

Thank you for improving the workflow Daniel!

Copy link
Member

@williln williln left a comment

Choose a reason for hiding this comment

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

Sorry I'm so behind on looking at this, but it looks great to me! Thank you @cunla!

Copy link
Member

@Stormheg Stormheg left a comment

Choose a reason for hiding this comment

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

Thanks Daniel - happy with this 👍

@cunla cunla merged commit 5622852 into main Nov 14, 2024
@cunla cunla deleted the new-proj-changes branch November 14, 2024 14:01
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.

5 participants