Skip to content

GH workflows for PR build and Releases#64

Merged
cijothomas merged 2 commits intoopen-telemetry:mainfrom
srprash:ci_workflows
Feb 9, 2021
Merged

GH workflows for PR build and Releases#64
cijothomas merged 2 commits intoopen-telemetry:mainfrom
srprash:ci_workflows

Conversation

@srprash
Copy link
Copy Markdown
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
Copy Markdown

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           

Comment thread .github/workflows/package-Exporter.Stackdriver.yml
@srprash
Copy link
Copy Markdown
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