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

opentelemetry-cpp project CI #2071

Merged
merged 5 commits into from
Jul 10, 2023
Merged

opentelemetry-cpp project CI #2071

merged 5 commits into from
Jul 10, 2023

Conversation

esigo
Copy link
Member

@esigo esigo commented Mar 26, 2023

Adds needs-triage label to new PRs and issues.
Also adds a comment to issues with help wanted label.

This would help us to identify items which needs maintainer reaction more easily.

For significant contributions please make sure you have completed the following items:

  • CHANGELOG.md updated for non-trivial changes
  • Unit tests have been added
  • Changes in public API reviewed

@codecov
Copy link

codecov bot commented Mar 26, 2023

Codecov Report

Merging #2071 (b13d9bd) into main (d6cebe1) will decrease coverage by 0.02%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2071      +/-   ##
==========================================
- Coverage   87.49%   87.47%   -0.02%     
==========================================
  Files         169      169              
  Lines        4891     4891              
==========================================
- Hits         4279     4278       -1     
- Misses        612      613       +1     

see 1 file with indirect coverage changes

@github-actions github-actions bot added the Stale label May 26, 2023
@esigo esigo self-assigned this Jun 19, 2023
@esigo esigo marked this pull request as ready for review July 8, 2023 12:20
@esigo esigo requested a review from a team July 8, 2023 12:20
@esigo esigo added the area:ci label Jul 8, 2023
Copy link
Member

@owent owent left a comment

Choose a reason for hiding this comment

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

LGTM and thanks.

Copy link
Member

@marcalff marcalff left a comment

Choose a reason for hiding this comment

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

LGTM.

Thanks for the automation.

@esigo esigo merged commit 5561393 into open-telemetry:main Jul 10, 2023
@esigo esigo deleted the cpp-project branch July 10, 2023 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants