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

refactor: split tracer setup code in its own part #25

Merged

Conversation

jd
Copy link
Member

@jd jd commented Dec 18, 2024

This will allow to replace the MergifyTracer dynamically in tests for
easier scenario testing.

@jd
Copy link
Member Author

jd commented Dec 18, 2024

This pull request is part of a stack:

  1. refactor: split tracer setup code in its own part (#25) 👈
  2. feat: capture OpenTelemetry logs (#26)

@jd jd mentioned this pull request Dec 18, 2024
Copy link

mergify bot commented Dec 18, 2024

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 Approval

Wonderful, this rule succeeded.
  • #approved-reviews-by >= 1

🟢 Continuous Integration

Wonderful, this rule succeeded.
  • all of:
    • check-success = test (3.10)
    • check-success = test (3.11)
    • check-success = test (3.12)
    • check-success = test (3.13)

🟢 Changelog requirements

Wonderful, this rule succeeded.
  • any of:
    • -title ~= ^feat
    • label = need changelog
    • label = skip changelog

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?:

🟢 🔎 Reviews

Wonderful, this rule succeeded.
  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

@jd jd force-pushed the devs/jd/capture-logs/Ib4124ff3dc885e609bb51b96f263b4733d5bc143 branch from a9a8311 to d2fd5ea Compare December 18, 2024 16:31
@jd jd force-pushed the devs/jd/capture-logs/I2de0216fc8a6ef4cca1f37d00e56e97a2552f2b6 branch 2 times, most recently from d800068 to 7c5c977 Compare December 18, 2024 16:45
Base automatically changed from devs/jd/capture-logs/Ib4124ff3dc885e609bb51b96f263b4733d5bc143 to main December 18, 2024 16:47
Copy link

mergify bot commented Dec 18, 2024

@jd this pull request is now in conflict 😩

@mergify mergify bot added the conflict label Dec 18, 2024
@jd jd force-pushed the devs/jd/capture-logs/I2de0216fc8a6ef4cca1f37d00e56e97a2552f2b6 branch from 7c5c977 to 429445c Compare December 18, 2024 16:51
@mergify mergify bot removed the conflict label Dec 18, 2024
@jd jd force-pushed the devs/jd/capture-logs/I2de0216fc8a6ef4cca1f37d00e56e97a2552f2b6 branch 4 times, most recently from 716c460 to 145d8b0 Compare December 18, 2024 17:15
@mergify mergify bot requested a review from a team December 18, 2024 17:16
This will allow to replace the MergifyTracer dynamically in tests for
easier scenario testing.

Change-Id: I2de0216fc8a6ef4cca1f37d00e56e97a2552f2b6
@jd jd force-pushed the devs/jd/capture-logs/I2de0216fc8a6ef4cca1f37d00e56e97a2552f2b6 branch from 145d8b0 to ed6a94b Compare December 18, 2024 17:17
@mergify mergify bot merged commit 8a73cb8 into main Dec 19, 2024
9 checks passed
@mergify mergify bot added the queued label Dec 19, 2024
@mergify mergify bot deleted the devs/jd/capture-logs/I2de0216fc8a6ef4cca1f37d00e56e97a2552f2b6 branch December 19, 2024 08:42
@mergify mergify bot removed the queued label Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants