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

Fix telemetry not support v1.0 #483

Merged
merged 1 commit into from
Aug 10, 2021

Conversation

wingyplus
Copy link
Contributor

The patch try to make this library support telemetry v1.0. Currently, it
doesn't because it contraint between [0.4.3, 1.0.0). Fix by using or
constraint to support both version.

@maennchen
Copy link
Member

Thanks for your PR @wingyplus ❤️

Can you correct the requirement so that for example 0.4.1 does not match?

@coveralls
Copy link

coveralls commented Jul 21, 2021

Pull Request Test Coverage Report for Build 201b3ca71b0ee6665dd8f1fe57bf9a65c38bef66-PR-483

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 2 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.3%) to 85.994%

Files with Coverage Reduction New Missed Lines %
lib/quantum/executor.ex 2 90.0%
Totals Coverage Status
Change from base Build 91a4df2490b9ebf01e6a46299920cd0550a89f62: -0.3%
Covered Lines: 307
Relevant Lines: 357

💛 - Coveralls

The patch try to make this library support telemetry v1.0. Currently, it
doesn't because it contraint between [0.4.3, 1.0.0). Fix by using `or`
constraint to support both version.
@wingyplus
Copy link
Contributor Author

@maennchen Oh, that's my mistake. We should constraint to ~> 0.4.3 or ~> 1.0 because quantum use :telemetry.span. The patch is fix.

@wingyplus
Copy link
Contributor Author

@maennchen Can we merge this one?

@maennchen maennchen merged commit 9b7787d into quantum-elixir:master Aug 10, 2021
@maennchen
Copy link
Member

@wingyplus Oops, sorry for the delay.

@maennchen
Copy link
Member

@wingyplus Released as v3.4.0

@wingyplus wingyplus deleted the fix-telemetry-1-0 branch August 10, 2021 08:30
@wingyplus
Copy link
Contributor Author

@maennchen Thank you. 🙇‍♂️

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.

4 participants