Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

chore(scripts/publish): get dist-tag from package.json #12722

Closed
wants to merge 1 commit into from

Conversation

petebacondarwin
Copy link
Contributor

This change would need to be cherry-picked to the previous version branches (with suitable change to package.json) to prevent releases from them breaking npm dist-tags.

Where does this need to be documented? New big version release tasks? Contributor documentation?

@petebacondarwin
Copy link
Contributor Author

The branch to dist-tag mapping will be as follows (once we fork and start working on 1.5.0):

  • 1.2.x -> ie8_compat
  • 1.3.x -> previous_1_3
  • 1.4.x -> latest
  • master -> beta

@Narretz
Copy link
Contributor

Narretz commented Sep 1, 2015

I think it's okay, although it's a bit cumbersome to update the branches.
Do we have access to our git in this step? Then we could check if there's a larger version number, and not set it in this case.

@Narretz Narretz added this to the 1.4.6 milestone Sep 1, 2015
@petebacondarwin
Copy link
Contributor Author

Going forward it will be quite straightforward. You only ever have to change the branch that is going from latest to previous... and then master swaps between latest and beta. We need to specify some dist-tag when publishing or it defaults to latest

@IgorMinar
Copy link
Contributor

nice! lgtm

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

Successfully merging this pull request may close these issues.

4 participants