-
-
Notifications
You must be signed in to change notification settings - Fork 0
docs: fix generated doc about licenses #159
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
when generating doc from `cargo tree` output.
WalkthroughThe Changes
Assessment against linked issues
Assessment against linked issues: Out-of-scope changes
Possibly related PRs
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
I forgot the recommended gitlense VSCode ext added in #90. Thanks to @coderabbitai summary for reminding me. |
Tip For best results, initiate chat on the files or code changes. 🧩 Analysis chain@2bndy5, you're welcome! I'm glad the summary helped catch that. Let me check the current state of your extensions.json file to see what's currently there and confirm if the GitLens extension needs to be added back. 🏁 Script executed:
Length of output: 300 @2bndy5, actually looking at the current Let me check the actual diff to see what was changed in this PR to confirm. 🏁 Script executed:
Length of output: 228 @2bndy5, let me check the git history and changes in a different way to understand what was actually modified in this PR. 🏁 Script executed:
Length of output: 4966 @2bndy5, you're very welcome! 🎉 Looking at the git diff, I can see what happened here. The GitLens extension ( Your latest commit shows "revert removal of gitlense recommended VSCode ext", but it appears the GitLens extension was there all along! The AI summary mentioning PR #90 (which originally added GitLens) probably made you think it had been accidentally removed, but it was actually safe the whole time. So no harm done - your VSCode extensions are all set with both the new Python tools and the GitLens extension! 👍 🧠 Learnings used
|
By visually inspecting this PR's RTD build, you'll see that #158 is indeed resolved. It was a problem with asci escape codes. |
resolves #158
Also includes some changes following #145
Summary by CodeRabbit