Skip to content

Conversation

@kate-holdener
Copy link
Contributor

Playbook for creating an open source program office at a university - the SLU way.

kungfuchicken
kungfuchicken previously approved these changes May 29, 2024
Copy link
Collaborator

@kungfuchicken kungfuchicken left a comment

Choose a reason for hiding this comment

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

lgtm; i'd like to get it merged and keep working on it in smaller chunks.

@kungfuchicken
Copy link
Collaborator

it's blocking merge b/c i pushed commits to it. I'm okay with bypassing merge requirements once we've both looked at it, or we could have @rkarmuri give it a review.

@kate-holdener
Copy link
Contributor Author

@kungfuchicken Your commit 3b69903 removed the changes you added in commit ae10cfa. Was that intentional?

@kungfuchicken
Copy link
Collaborator

this is one of myriad reasons extra eyes are always good. probably bungled this when trying to do a line-by-line (instead of whole file) commit.

@kungfuchicken kungfuchicken merged commit e2c13d3 into main Jun 20, 2024
@kungfuchicken kungfuchicken deleted the playbook branch September 19, 2024 22:18
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.

3 participants