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

publish to all the registries listed in oc.json #52

Merged
merged 1 commit into from
Apr 21, 2015
Merged

publish to all the registries listed in oc.json #52

merged 1 commit into from
Apr 21, 2015

Conversation

andyroyle
Copy link
Collaborator

Run async.eachSeries over the list of registry urls and publish the package to each one.

Currently it ignores any errors (lets the logger print them to the console), and continues publishing even if a previous publish failed.

@andyroyle
Copy link
Collaborator Author

ah feck just realised that i pushed to opentable, rather than my fork. DOH!

@matteofigus
Copy link
Member

No worries, just put a don't merge on the title and I'll keep it there until you are happy with it

@andyroyle
Copy link
Collaborator Author

this is good to go, just as soon as I work out why the tests are failing

@matteofigus
Copy link
Member

Looks like SauceLabs is having troubles - re-pushed the button.

@matteofigus
Copy link
Member

I tried to make some experiments to stabilise SauceLabs. if you take the latest changes from the master, we can have another go and see if it goes green @andyroyle ? Thanks

@andyroyle
Copy link
Collaborator Author

rebased, lets see if it works this time

@matteofigus
Copy link
Member

Ah, apparently Travis doesn't correctly use proper credentials when using encrypted environment variables coming from forks. I made you admin, next time just create a new branch here 👍

matteofigus added a commit that referenced this pull request Apr 21, 2015
publish to all the registries listed in oc.json
@matteofigus matteofigus merged commit e486cdc into opencomponents:master Apr 21, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants