Skip to content

refactor: shorten link to sibling document - #296

Merged
danielleroux merged 1 commit into
mainfrom
refactor/link-in-folder
Sep 9, 2026
Merged

refactor: shorten link to sibling document#296
danielleroux merged 1 commit into
mainfrom
refactor/link-in-folder

Conversation

@timowolf

@timowolf timowolf commented Sep 7, 2026

Copy link
Copy Markdown
Member

No need to go up to language as target is in same folder. This breaks when copying the ux writing guide into different folder.

💡 What is the current behavior?

GitHub Issue Number: #

🆕 What is the new behavior?

👨‍💻 Help & support

No need to go up to language as target is in same folder.
This breaks when copying the ux writing guide into different
folder.
@timowolf
timowolf requested review from a team as code owners September 7, 2026 16:06
@netlify

netlify Bot commented Sep 7, 2026

Copy link
Copy Markdown

Deploy Preview for industrial-experience ready!

Name Link
🔨 Latest commit 1b06893
🔍 Latest deploy log https://app.netlify.com/projects/industrial-experience/deploys/6a9ee1039a3afc00083b5f74
😎 Deploy Preview https://deploy-preview-296--industrial-experience.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai

coderabbitai Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Team

Run ID: 496b7eac-11f6-4ac3-9de1-16c8e16f9be0

📥 Commits

Reviewing files that changed from the base of the PR and between 8e9394f and 1b06893.

📒 Files selected for processing (1)
  • docs/guidelines/language/writing-style-guide-getting-started.md

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The writing style guide now uses a local relative path for the “Frequent app functions” link.

Changes

Documentation Link Correction

Layer / File(s) Summary
Update the documentation link
docs/guidelines/language/writing-style-guide-getting-started.md
The “Frequent app functions” link now targets ./frequent-app-functions.md.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 1b068

The guide now links directly to the sibling Frequent app functions document, avoiding breakage when copied. The change is ready to merge.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the change to shorten the link to a sibling document. It is concise and specific.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch refactor/link-in-folder

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.

@danielleroux
danielleroux merged commit 5102b4a into main Sep 9, 2026
6 checks passed
@danielleroux
danielleroux deleted the refactor/link-in-folder branch September 9, 2026 07:16
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.

2 participants