Skip to content

GH2193: Added new maintainers#2194

Merged
daveaglick merged 1 commit intocake-build:developfrom
gep13:feature/GH-2193
Jun 25, 2018
Merged

GH2193: Added new maintainers#2194
daveaglick merged 1 commit intocake-build:developfrom
gep13:feature/GH-2193

Conversation

@gep13
Copy link
Copy Markdown
Member

@gep13 gep13 commented Jun 17, 2018

Think I have got all the required places

@gep13 gep13 requested review from daveaglick and pascalberger June 17, 2018 10:42
@gep13
Copy link
Copy Markdown
Member Author

gep13 commented Jun 17, 2018

Relates to #2193

@gep13
Copy link
Copy Markdown
Member Author

gep13 commented Jun 17, 2018

@pascalberger if you are happy with this change, then you can feel free to merge it. For a larger, code related change, we might need a couple of thumbs up, but for this, I think we are good to go.

@gep13
Copy link
Copy Markdown
Member Author

gep13 commented Jun 17, 2018

The process that we follow for Cake is going to be similar to what you already follow for your other repos, but, main things would be:

  • there is an associated issue created, and referenced in the PR
  • issue is labelled and assigned to a milestone
  • PR targets the develop branch (unless part of a hotfix)
  • ideally PR only has one commit, unless compelling reason to separate them out
  • Commit messages reference the issue using (GH-xxx) nomenclature
  • CLA has been signed
  • all CI checks pass
  • PR comes from another branch on fork, i.e. not from develop branch

Let me know if you have any questions.

@devlead devlead changed the title (GH-2193) Added new maintainers GH2193: Added new maintainers Jun 20, 2018
Copy link
Copy Markdown
Member

@devlead devlead left a comment

Choose a reason for hiding this comment

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

@gep13 LGTM 👍

@pascalberger & @daveaglick agree?

@daveaglick
Copy link
Copy Markdown
Member

I've been stalling on my review because honestly, I was hoping to be listed as CodeNinja McGooglyEyes. I guess this will suffice though.

@daveaglick daveaglick merged commit ddb5835 into cake-build:develop Jun 25, 2018
@gep13 gep13 deleted the feature/GH-2193 branch June 25, 2018 15: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.

5 participants