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

NH-55892 upload workflow #1

Merged
merged 6 commits into from
Dec 12, 2023
Merged

NH-55892 upload workflow #1

merged 6 commits into from
Dec 12, 2023

Conversation

xuan-cao-swi
Copy link

No description provided.

@xuan-cao-swi xuan-cao-swi requested a review from a team November 30, 2023 18:05
Copy link

@cheempz cheempz left a comment

Choose a reason for hiding this comment

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

Same comments as for solarwinds/opentelemetry-ruby-contrib#1 apply to this PR :)

@xuan-cao-swi xuan-cao-swi requested a review from a team December 6, 2023 19:26

# build and push gem
gem build "opentelemetry-$GEM_NAME.gemspec"
# gem push --key github --host https://rubygems.pkg.github.com/solarwinds "opentelemetry-instrumentation-$GEM_NAME-$gem_version.gem"

Choose a reason for hiding this comment

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

Is this line needed?

Copy link
Author

Choose a reason for hiding this comment

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

Yes, just added it back.


GEM_NAME="$1"

case $GEM_NAME in
Copy link

@tammy-baylis-swi tammy-baylis-swi Dec 11, 2023

Choose a reason for hiding this comment

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

Nice that we are able to have a map here 👍 compared to solarwinds/opentelemetry-ruby-contrib#1

@tammy-baylis-swi
Copy link

Thanks @xuan-cao-swi ! Just one more suggestion.

Copy link

@tammy-baylis-swi tammy-baylis-swi left a comment

Choose a reason for hiding this comment

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

Thanks again!

@xuan-cao-swi xuan-cao-swi merged commit 556c02b into main Dec 12, 2023
50 of 51 checks passed
@xuan-cao-swi xuan-cao-swi deleted the package-workflow branch December 12, 2023 14:41
@cheempz cheempz changed the title upload workflow NH-55892 upload workflow Dec 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants