Skip to content
This repository was archived by the owner on Jan 11, 2023. It is now read-only.

Conversation

@vishrutshah
Copy link
Contributor

Swallowing errors while publishing to allow publishing of the only new version of the gems

Copy link
Contributor

@veronicagg veronicagg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving as a temporary fix for recent publish transient error.

# Using execute method so that keys are not logged onto console
execute("gem push ./pkg/#{dir}-#{version}.gem" + (args[:key].nil? ? '' : " -k #{args[:key]}"))
rescue Exception => ex
puts "Error occurred while publishing #{dir}: #{ex}"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would do caps on "ERROR"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that's true but i'll revert this changes once we release :)

@vishrutshah
Copy link
Contributor Author

Thanks for the quick review!!

@vishrutshah vishrutshah merged commit 59eeea5 into Azure:master Sep 12, 2017
@vishrutshah vishrutshah deleted the rescue-publish branch September 12, 2017 01:03
vishrutshah added a commit that referenced this pull request Sep 12, 2017
vishrutshah added a commit to vishrutshah/azure-sdk-for-ruby that referenced this pull request Sep 12, 2017
vishrutshah added a commit that referenced this pull request Sep 12, 2017
vishrutshah added a commit to vishrutshah/azure-sdk-for-ruby that referenced this pull request Sep 27, 2017
vishrutshah added a commit that referenced this pull request Sep 27, 2017
vishrutshah added a commit that referenced this pull request Sep 28, 2017
vishrutshah added a commit that referenced this pull request Sep 28, 2017
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.

3 participants