-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Trigger canary release on successful PR build #246
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
56 commits
Select commit
Hold shift + click to select a range
bf8942a
Setting up after_success for travisci
jonrohan 2e44d06
removing quotes
jonrohan a572720
testing with deploy script
jonrohan 9ce110d
Change provider to script
jonrohan 50e6f4c
trying with an extra env
jonrohan fa38af4
on all branches
jonrohan a20f259
Change to the build directory
jonrohan 29e2566
Constructing version name
jonrohan bdc537f
removing npm script for now
jonrohan a41446a
Using short commit
jonrohan 8f443ab
Using matrix
jonrohan c1b119a
Abstracting out module name
jonrohan 570b667
Creating a deploy matrix
jonrohan 5bb4059
Don't deploy on dev or master
jonrohan 7613846
let's do this
jonrohan 95aac5b
removing global commit variable and installing dev dependencies
jonrohan b8d7aeb
Running script
jonrohan b482fd8
Sort these so we don't overlook any
jonrohan 610e213
creating a dummy script to avoid failed deploy
jonrohan 79cf276
Changing permissions on script
jonrohan af83807
update publish scripts
2b16f69
add top-level test script
e5a115e
just echo for now
2f20282
add basic branch logic
14c5ecb
rename 'publish' => 'release'; don't run recursively
908ab0e
stub out CI deployment logic
f71f97d
Merge branch 'dev' into after_success
shawnbot 172f6c2
remove top-level test script for now
88a0b4e
rename to deploy
jonrohan a4d6392
rename to deploy
jonrohan 999c6d4
skipping deploy
jonrohan 65925c3
add back npm test script
89ba9a0
try testing from top level
4abd9bd
try bash
jonrohan cf82ee4
Use travis event type
jonrohan 4bb84f8
rename deploy => cibuild
shawnbot 03d2e1f
run all ci release commands with --yes
shawnbot e18e8f7
remove --skip-git lerna arg
shawnbot 0a07d2f
upgrade to lerna@2.0.0
shawnbot e99ae79
authenticate npm w/NPM_TOKEN env
shawnbot c3abddc
call script/cibuild with --yes
shawnbot f40a30c
get ci_args from argv[1:]
shawnbot 0d3f6b7
rename ci_args to args
shawnbot 9cce630
use NPM_API_KEY instead of NPM_TOKEN
shawnbot d96aa3b
set --npm-tag=canary for release-canary script
shawnbot 92656aa
add a comment to explain why we're passing args
shawnbot 53ec496
add compare-published script
shawnbot c1ad972
tidy up compare-published script
shawnbot 9991349
add caveat in compare-published
shawnbot d7ddfef
reorder lerna publish args
shawnbot b57d529
echo the rc and release commands, just to be safe
shawnbot 871d806
disable primer/selector-no-utility
shawnbot 7659b63
confirm travis auth
shawnbot ae023e0
try npm config set
shawnbot a9559e8
run stuff again
shawnbot 0ce097e
nix `npm whoami` call
shawnbot File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Using matrix
- Loading branch information
commit 8f443ab786b66d05708a090ce681ee1aaa498ff2
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hrm, does this modify the build matrix? If so, maybe we just do this when we generate the prerelease id.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think it changed the matrix, https://travis-ci.org/primer/primer-css/builds/251305487
Looks like it just included a
$COMMITvariable in all the builds