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: subscriber context docs #2894

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kaffarell
Copy link
Contributor

In the documentation of the layer context span_scope method, the note contained a reference to a scope() method, which was removed some time ago. Also fixed a phrasing error above.

Fixes: #2890

@kaffarell kaffarell requested review from hawkw and a team as code owners February 29, 2024 12:54
@hawkw
Copy link
Member

hawkw commented Mar 1, 2024

Hi @kaffarell, thanks for the fix! I believe that this issue also exists in the documentation on the master branch. Would you mind changing this PR to be based on the master branch? We prefer to merge changes that apply to both the v0.1.x and master branches to master, and then backport them to v0.1.x. This helps ensure that bugs fixed in the release version are also fixed in v0.2. Thank you!

@kaffarell kaffarell changed the base branch from v0.1.x to master March 4, 2024 08:18
@kaffarell kaffarell closed this Mar 4, 2024
In the documentation of the layer context span_scope method, the note
contained a reference to a `scope()` method, which was removed some time
ago. Also fixed a phrasing error above.

Fixes: tokio-rs#2890
@kaffarell kaffarell reopened this Mar 4, 2024
@kaffarell
Copy link
Contributor Author

Oops, didn't mean to close it :)
Anyway, the base branch should now be master.

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.

Context docs in tracing-subscriber reference scope method that does not exist
2 participants