[chef-dev] Re: Anything other than CLA and pull request to get changes merged?


Chronological Thread 
  • From: Bryan McLellan < >
  • To: Christopher Armstrong < >
  • Cc: Chef Dev < >
  • Subject: [chef-dev] Re: Anything other than CLA and pull request to get changes merged?
  • Date: Thu, 24 Oct 2013 12:21:15 -0400

That's it Chris, thanks!

The basic steps are here: http://docs.opscode.com/community_contributions.html

Cookbook Code Review happens Monday, Wednesday and Friday, as
documented on the Code Review link that Kannan pointed out.

I see you've got a commit on your pull request to fix up an change.
It's worth looking into using "git rebase -i" to merge that commit
back into a single commit, and add the ticket number to the commit
message. That helps other contributors down the road find more
information about what a certain change was made by easily being able
to find the ticket.

It's good when you bump a version to explicitly mention on the ticket
that you've tested this version on such and such platform, and the
reason the new version is important is because of foo security fix or
bar awesome feature.

Bryan



Archive powered by MHonArc 2.6.16.

§