Skip to content

Remove doc specifying future anchor support #2375

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

Conversation

AnthonyRonning
Copy link
Contributor

I believe it's fully supported now as of this next release, right? Just came across this comment.

@codecov-commenter
Copy link

codecov-commenter commented Jun 25, 2023

Codecov Report

Patch coverage has no change and project coverage change: +0.27 🎉

Comparison is base (d327c23) 90.32% compared to head (5cc625e) 90.59%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2375      +/-   ##
==========================================
+ Coverage   90.32%   90.59%   +0.27%     
==========================================
  Files         106      106              
  Lines       54948    56882    +1934     
  Branches    54948    56882    +1934     
==========================================
+ Hits        49630    51534    +1904     
- Misses       5318     5348      +30     
Impacted Files Coverage Δ
lightning/src/events/mod.rs 41.86% <ø> (ø)

... and 5 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@TheBlueMatt
Copy link
Collaborator

Good catch. Can you link to the config option to turn on anchors and say that its not generated if that option isnt on?

@AnthonyRonning AnthonyRonning force-pushed the remove-doc-future-anchor-support branch 2 times, most recently from 471ca99 to eeb06a3 Compare June 29, 2023 18:57
@AnthonyRonning
Copy link
Contributor Author

Good catch. Can you link to the config option to turn on anchors and say that its not generated if that option isnt on?

Added. Let me know if there's any nits there.

@TheBlueMatt TheBlueMatt added this to the 0.0.116 milestone Jun 30, 2023
@AnthonyRonning AnthonyRonning force-pushed the remove-doc-future-anchor-support branch from eeb06a3 to 5cc625e Compare June 30, 2023 03:10
@TheBlueMatt TheBlueMatt merged commit 63c77a9 into lightningdevkit:main Jul 2, 2023
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