Skip to content

fix: subscriber context docs #2894

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

Merged
merged 2 commits into from
May 25, 2024

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
@kaffarell kaffarell force-pushed the subscriber_context_docs branch from ca89c3c to 908cc43 Compare March 4, 2024 08:30
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.

@davidbarsky davidbarsky enabled auto-merge (squash) May 25, 2024 20:53
@davidbarsky davidbarsky merged commit 382ee01 into tokio-rs:master May 25, 2024
54 of 55 checks passed
@hds hds mentioned this pull request Nov 19, 2024
20 tasks
hds pushed a commit that referenced this pull request Nov 21, 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: #2890
hds pushed a commit that referenced this pull request Nov 22, 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: #2890
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
4 participants