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

GH workflows for PR build and Releases #64

Merged
merged 2 commits into from
Feb 9, 2021

Conversation

srprash
Copy link
Contributor

@srprash srprash commented Feb 7, 2021

Changes:

  • Combining separate PR build workflows for windows and linux into one.
  • Adding workflows for releasing projects on tag creation event.

Probably it's not the best approach to have a separate release workflow for each project because there's a lot of code duplication. But so far I couldn't find a way to reuse or include one common workflow into specific ones. Happy to discuss a better way to do this.

@srprash srprash requested a review from a team February 7, 2021 23:10
@codecov
Copy link

codecov bot commented Feb 7, 2021

Codecov Report

Merging #64 (45fe94b) into main (736e6c0) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #64   +/-   ##
=======================================
  Coverage   70.95%   70.95%           
=======================================
  Files          28       28           
  Lines         809      809           
=======================================
  Hits          574      574           
  Misses        235      235           

@srprash
Copy link
Contributor Author

srprash commented Feb 9, 2021

We should also remove the common pack and upload to myget workflow: https://github.com/open-telemetry/opentelemetry-dotnet-contrib/blob/main/.github/workflows/dotnet-core.yml
since it runs on any tag creation and nightly.

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.

3 participants