Skip to content

Fix broken link#2256

Merged
KaveeshaPiumini merged 1 commit intomainfrom
jeradrutnam-patch-2
Apr 8, 2026
Merged

Fix broken link#2256
KaveeshaPiumini merged 1 commit intomainfrom
jeradrutnam-patch-2

Conversation

@jeradrutnam
Copy link
Copy Markdown
Contributor

@jeradrutnam jeradrutnam commented Apr 8, 2026

This pull request updates the documentation link in the README.md file to point to the latest "Getting Started" guide for Thunder. This ensures that users are directed to the most up-to-date onboarding material.

Documentation update:

  • Updated the documentation link in README.md to reference the "What is Thunder" guide under the /next/guides/getting-started/ path.

Summary by CodeRabbit

  • Documentation
    • Updated documentation link to point to the new getting-started guide.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 8, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 994e2f00-c07b-42ac-b6ba-b68bb590875c

📥 Commits

Reviewing files that changed from the base of the PR and between a233ccf and 6bddd69.

📒 Files selected for processing (1)
  • README.md

📝 Walkthrough

Walkthrough

Updated a documentation link in the README from the legacy introduction page to a new "what is Thunder" getting-started guide page. No code changes or structural modifications were made.

Changes

Cohort / File(s) Summary
Documentation Link Update
README.md
Updated the Documentation hyperlink URL from /docs/guides/introduction to /docs/next/guides/getting-started/what-is-thunder.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

skip-changelog

Suggested reviewers

  • DonOmalVindula
  • brionmario
🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description lacks required template sections including Purpose (adequate but brief), Approach, and most of the Checklist items. While it explains the change, it does not follow the repository's template structure. Complete the pull request description template by adding Purpose, Approach, Checklist items, and Security checks sections to align with repository requirements.
Title check ❓ Inconclusive The title 'Fix broken link' is vague and generic. While it relates to the change (updating a documentation link), it lacks specificity about what link or why it needed fixing. Use a more descriptive title such as 'Update documentation link to latest getting-started guide' to clearly convey the purpose of the change.
✅ Passed checks (1 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jeradrutnam-patch-2

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

⚠️ Deprecation Warning: The deny-licenses option is deprecated for possible removal in the next major release. For more information, see issue 997.

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 6bddd69.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.51%. Comparing base (a233ccf) to head (6bddd69).
⚠️ Report is 15 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2256   +/-   ##
=======================================
  Coverage   89.51%   89.51%           
=======================================
  Files         911      911           
  Lines       60145    60145           
=======================================
  Hits        53839    53839           
  Misses       4686     4686           
  Partials     1620     1620           
Flag Coverage Δ
backend-integration-postgres 53.37% <ø> (ø)
backend-integration-sqlite 53.37% <ø> (-0.04%) ⬇️
backend-unit 83.95% <ø> (ø)
frontend-apps-console-unit 90.14% <ø> (ø)
frontend-apps-gate-unit 97.48% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@KaveeshaPiumini KaveeshaPiumini added Type/Docs skip-changelog Skip generating changelog for a particular PR labels Apr 8, 2026
@KaveeshaPiumini KaveeshaPiumini added this pull request to the merge queue Apr 8, 2026
Merged via the queue into main with commit b8fe554 Apr 8, 2026
33 of 34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog Skip generating changelog for a particular PR Type/Docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants