citgm icon indicating copy to clipboard operation
citgm copied to clipboard

Enable testing a Github branch other than master

Open gibfahn opened this issue 8 years ago • 4 comments

It is currently possible to use citgm to test the master branch with the following lookup.json options:

"master": true,
"repo": "https://github.com/org/repo",
"replace": true

However, there's no way to select a different branch, which makes it harder to use CitGM for development.

The fix would be to have the option "branch": "my_branch" instead of "master": true. We can alias "master": true to "branch": "master" for backwards compatibility.

gibfahn avatar Dec 15 '16 09:12 gibfahn

You can use a branch with this option:

"sha": "<git-commit-sha>"    Test against a specific commit

I think you still need "master": true.

richardlau avatar Dec 15 '16 14:12 richardlau

So, using this lookup.json:

{
  "nodereport": {
    "master": true,
    "repo": "https://github.com/richardlau/nodereport",
    "sha": "build",
    "replace": true
  }
}

And running citgm-all -v verbose -l lookup.json nodereport you can indeed run CitGM against a branch. However, the equivalent citgm command (citgm -c build -v verbose richardlau/nodereport) doesn't work.

gibfahn avatar Dec 15 '16 14:12 gibfahn

you can add a hash after the name to reference the branch.

On Thu, Dec 15, 2016, 9:56 AM Gibson Fahnestock [email protected] wrote:

So, using this lookup.json:

{ "nodereport": { "master": true, "repo": "https://github.com/richardlau/nodereport", "sha": "build", "replace": true } }

And running citgm-all -v verbose -l lookup.json nodereport you can indeed run CitGM against a branch. However, the equivalent citgm command (citgm -c build -v verbose richardlau/nodereport) doesn't work.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/nodejs/citgm/issues/279#issuecomment-267347298, or mute the thread https://github.com/notifications/unsubscribe-auth/AAecV-m9z5PUdtjPjpoefXDxBAI68Hzkks5rIVUdgaJpZM4LN4ER .

MylesBorins avatar Dec 15 '16 14:12 MylesBorins

Oh yeah, so what does the -c option do?

gibfahn avatar Dec 15 '16 15:12 gibfahn