Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

allow packages to be installed from a tag #171

Closed
bcoe opened this issue Aug 25, 2016 · 11 comments
Closed

allow packages to be installed from a tag #171

bcoe opened this issue Aug 25, 2016 · 11 comments
Assignees
Milestone

Comments

@bcoe
Copy link

bcoe commented Aug 25, 2016

It would be neat if the modules in the list to be smoke-tested could be published to a tag.

This would allow me to publish fixes for yargs to a next tag, so that you can validate that the Canary tests are running appropriately, without having to cut a full release of the project.

@bengl
Copy link
Member

bengl commented Sep 20, 2016

You can at least use the sha CLI param (or lookup.json property), but at the moment you still have to change that on each publish.

@MylesBorins MylesBorins added this to the v2.0.0 milestone Jan 4, 2017
@gdams
Copy link
Member

gdams commented Jan 9, 2017

I'll take a look and see if I can get citgm working with a custom tag if that's what you're asking for

@gdams gdams self-assigned this Jan 9, 2017
gdams pushed a commit to gdams/citgm that referenced this issue Jan 9, 2017
this PR allows packages to be installed from a tag using
`citgm <module> -T <version>`.
Fixes: nodejs#171
gdams pushed a commit to gdams/citgm that referenced this issue Jan 9, 2017
this PR allows packages to be installed from a tag using
`citgm <module> -T <version>`.
Fixes: nodejs#171
@gibfahn
Copy link
Member

gibfahn commented Jan 9, 2017

I'm pretty sure you can do something like:

{
  "yargs": {
    "master": true,
    "repo": "https://github.com/yargs/yargs",
    "sha": "next"
  }
}

in the lookup.json, or citgm yargs/yargs#next

It shoudn't really be necessary to specify master and repo (we're not actually getting it from master, and the repo should be in the package.json), but that's what seems to work atm.

See #279 (comment)

@MylesBorins
Copy link
Contributor

Perhaps we could rename sha to head and deprecate master.

@gibfahn
Copy link
Member

gibfahn commented Jan 10, 2017

I think "master": true is basically the same "sha": "master", so we could definitely deprecate master. However I prefer sha to head (although maybe commit would be best), as what you provide is a <commit-ish>.

@MylesBorins
Copy link
Contributor

putting on my helmet before going in the bike shed...

isn't whatever reference you are giving the head of the tree you want to use? master, some-branch, Some-Sha are all heads.

¯\_(ツ)_/¯

@gibfahn
Copy link
Member

gibfahn commented Jan 10, 2017

Sure, but you might just directly pass a commit like cf004c0 that isn't the head of a tree right? Maybe it is, I'm not that familiar with git terminology, but I always thought that HEAD was just the currently checked out commit,

More to the point, if I just saw head as an option in the lookup.json I wouldn't see it as obviously as a reference to a git commit compared to either commit or sha.

@MylesBorins
Copy link
Contributor

MylesBorins commented Jan 10, 2017 via email

@MylesBorins
Copy link
Contributor

ACTUALLY... a specific sha is not a head... https://git-scm.com/book/en/v2/Git-Internals-Git-References

@gdams gdams assigned richardlau and unassigned gdams Mar 2, 2017
@gdams
Copy link
Member

gdams commented Mar 2, 2017

@richardlau would you be able to take a look at this when you get a chance ?

@targos
Copy link
Member

targos commented May 18, 2019

Closing as I believe the sha option is enough for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants