opentelemetry-ruby icon indicating copy to clipboard operation
opentelemetry-ruby copied to clipboard

Better: remove restrictions on koala.

Open ylecuyer opened this issue 2 years ago • 3 comments

Following https://github.com/arsduo/koala/issues/659 and the recent release of koala https://github.com/arsduo/koala/pull/663, restrictions on koala and faraday can be removed

ylecuyer avatar Jan 18 '22 22:01 ylecuyer

CLA Not Signed

@ylecuyer hey ,sorry been a bit underwater, not sure why the CLA isn't taking, mind pushing an empty commit to kick the CLA check?

ericmustin avatar Feb 22 '22 14:02 ericmustin

Hello, and thank you for your contribution!

We recently split Ruby instrumentation out into the opentelemetry-ruby-contrib repo.

This PR is related to instrumentation, so we'll need you to re-open it against opentelemetry-ruby-contrib. Sorry for the inconvenience!

To do that, you can:

  1. Create a fork of opentelemetry-ruby-contrib and copy the git url
  2. In your opentelemetry-ruby repo, run git remote add tmp-contrib <your-fork-git-url>
  3. git push tmp-contrib your-branch-name
  4. Open a new PR in contrib (feel free to just copy/paste your original PR description there)
  5. Close your open PR in this repo with a comment that links to your new PR in contrib
  6. Delete your tmp-contrib remote from opentelemetry-ruby (git remote rm tmp-contrib)
  7. git clone your opentelemetry-ruby-contrib fork, check out your branch, and make all changes in that repo from now on!

Sorry again for the inconvenience, and thank you for contributing!

plantfansam avatar Jun 17 '22 17:06 plantfansam

👋 This pull request has been marked as stale because it has been open with no activity. You can: comment on the issue or remove the stale label to hold stale off for a while, add the keep label to hold stale off permanently, or do nothing. If you do nothing this pull request will be closed eventually by the stale bot

github-actions[bot] avatar Mar 31 '24 01:03 github-actions[bot]

👋 Hi, @ylecuyer! Thank you for your PR!

As Sam mentioned, Ruby instrumentation now lives in the opentelemetry-ruby-contrib repo. We'd be happy to take another look at your work in that context.

Since this PR is focused on instrumentation, I'm going to close it.

We appreciate your contribution and hope to work with you again soon!

kaylareopelle avatar Apr 01 '24 21:04 kaylareopelle