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

Add files to expose the currently used external types. #1303

Merged
merged 1 commit into from
Oct 23, 2023

Conversation

hdost
Copy link
Contributor

@hdost hdost commented Oct 17, 2023

To get ready for release candidate, we should keep track of these as it affects how stable we can consider being.

Changes

Catalogs currently used External Types.

Merge requirement checklist

  • CONTRIBUTING guidelines followed
  • Unit tests added/updated (if applicable)
  • Appropriate CHANGELOG.md files updated for non-trivial, user-facing changes
  • Changes in public API reviewed (if applicable)

Sorry, something went wrong.

@hdost hdost requested a review from a team October 17, 2023 16:41
@hdost hdost force-pushed the document-external-types branch from c6d328c to eb15768 Compare October 17, 2023 16:47
@codecov
Copy link

codecov bot commented Oct 17, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

see 1 file with indirect coverage changes

📢 Thoughts on this report? Let us know!.

Copy link
Contributor

@TommyCpp TommyCpp left a comment

Choose a reason for hiding this comment

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

Ah I was wondering how to guard our public API. Thanks for it!

Copy link
Contributor

@TommyCpp TommyCpp left a comment

Choose a reason for hiding this comment

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

Block this PR for now as we get feedback to add the current setup as CI

@hdost hdost force-pushed the document-external-types branch 3 times, most recently from 9fb2e60 to a177573 Compare October 23, 2023 20:31
To get ready for release candidate, we should keep track of these as it
affects how stable we can consider being.

Add check to CI.
@hdost hdost force-pushed the document-external-types branch from a177573 to 66c4b45 Compare October 23, 2023 20:35
@hdost
Copy link
Contributor Author

hdost commented Oct 23, 2023

Block this PR for now as we get feedback to add the current setup as CI

https://github.com/open-telemetry/opentelemetry-rust/actions/runs/6618694903/job/17977650680 << Added to CI, and verified that it will fail if we don't have something in the list.
We can always add more crates, but for now I figured we focus on API and SDK.

@hdost hdost requested a review from TommyCpp October 23, 2023 20:36
@TommyCpp
Copy link
Contributor

Perfect! Thanks for setting this up. I agree this is be enough for now

@TommyCpp TommyCpp merged commit 28d8cf9 into open-telemetry:main Oct 23, 2023
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

5 participants