-
Notifications
You must be signed in to change notification settings - Fork 26.8k
chore(ci): cleanup artifact upload #6232
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
Conversation
|
The build only job is also used for payload size analytics and continuous delivery to npm and pub. Can you keep it and just remove the upload to gcs? |
f743af7 to
7eace98
Compare
|
Sure, done. |
|
discussed offline... let's move the work to other works in a new PR. Can you also create an issue to audit our CI jobs and document what's On Mon, Jan 4, 2016 at 9:28 AM Alex Eagle notifications@github.com wrote:
|
|
filed #6256 for the follow-up |
14de4fc to
4859277
Compare
This is no longer needed for g3sync.
4859277 to
1257fd5
Compare
|
ready to merge |
|
Merging PR #6232 on behalf of @alxhub to branch presubmit-alxhub-pr-6232. |
|
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
This is no longer needed for g3sync. Also, it frees up a travis worker.