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

Migrate CircleCI ci.yml workflow to GitHub Actions #476

Merged
merged 6 commits into from
Dec 17, 2020

Conversation

AzfaarQureshi
Copy link
Contributor

What does this do

This addresses issue open-telemetry/opentelemetry-go#880 by migrating CI/CD from CircleCI to GitHub Actions

Changes

  • Migrated current-go, prior-go and integration-test CircleCI jobs to GitHub Actions.
  • Changelog updated

Migration Plan

I suggest having CircleCI and GitHub Action jobs run in parallel for a few weeks. After the GitHub Actions jobs are running fine for a week or so, remove the CircleCI workflows from config.yml

cc- @alolita

@AzfaarQureshi
Copy link
Contributor Author

AzfaarQureshi commented Dec 16, 2020

@MrAlias can you review/merge this too when you get a chance 😄

@codecov
Copy link

codecov bot commented Dec 16, 2020

Codecov Report

❗ No coverage uploaded for pull request base (master@e484e9e). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             master   #476   +/-   ##
=======================================
  Coverage          ?      0           
=======================================
  Files             ?      0           
  Lines             ?      0           
  Branches          ?      0           
=======================================
  Hits              ?      0           
  Misses            ?      0           
  Partials          ?      0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e484e9e...b3b419c. Read the comment docs.

@MrAlias MrAlias merged commit 31b2ca6 into open-telemetry:master Dec 17, 2020
@MrAlias MrAlias mentioned this pull request Jan 6, 2021
MrAlias pushed a commit that referenced this pull request Aug 4, 2021
These have been "skipped" since PR #476 was merged due to the lack of
setting the `INTEGRATION` environment variable prior to running the test.
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.

None yet

4 participants