Skip to content

Fix : Broken links for interview section#115

Merged
RAJ8664 merged 1 commit intomainfrom
link_fix
Mar 7, 2026
Merged

Fix : Broken links for interview section#115
RAJ8664 merged 1 commit intomainfrom
link_fix

Conversation

@RAJ8664
Copy link
Owner

@RAJ8664 RAJ8664 commented Mar 7, 2026

👋 Hey there! Thank you for contributing to Prep 🚀
Please fill out this pull request template carefully to help us review your changes quickly.

Please check all that apply: Place an "x" in the corresponding checkbox

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • 🚀 New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 🧹 Code cleanup or refactor
  • 🧪 Tests added or updated
  • 📘 Documentation update

Broken links under interview section are corrected.

fixes #114

mermaid diagrams has already been fixed as per issue #114

Ref -> 1b015a6#diff-7dcf1a2b8e42b3a406a29f3ad225b4ad3c37d1af701e034660b3f591a9e5747e

Summary by CodeRabbit

  • Documentation
    • Enhanced interview experience articles with improved formatting, readability, and consistency across multiple company interview guides.
    • Converted reference links to structured markdown format for better navigation.
    • Reorganized content sections and bullet lists for clearer presentation.
    • Minor wording refinements and standardized styling throughout the documentation.

@vercel
Copy link

vercel bot commented Mar 7, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
prep Ready Ready Preview, Comment Mar 7, 2026 10:13pm

@coderabbitai
Copy link

coderabbitai bot commented Mar 7, 2026

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

This PR applies formatting, link standardization, and editorial updates across 36+ interview experience documentation files spanning multiple companies including Amazon, Cisco, Google, Microsoft, Optum, and others. Changes include converting plain URLs to markdown links, reflowing bullet lists, adjusting emphasis styling, and minor punctuation refinements—all presentational without altering content structure or meaning.

Changes

Cohort / File(s) Summary
Amazon Interview Experiences
Interview/Amazon/Amazon_Exp1/index.md, Interview/Amazon/Amazon_Exp2/index.md
Formatting and punctuation adjustments; reflowed lists and spacing; minor wording consistency updates; no changes to interview process or outcomes.
Atlassian Interview Experience
Interview/Atlassian/Atlassian_Exp1/index.md
Added blank lines and bullet points for readability; converted plain URLs to markdown links; minor textual additions under interview rounds; presentation-level changes only.
Cisco Interview Experiences
Interview/Cisco/Cisco_Exp1/index.md, Interview/Cisco/Cisco_Exp2/index.md, Interview/Cisco/Cisco_Exp3/index.md, Interview/Cisco/Cisco_Exp4/index.md
Converted plain reference URLs to markdown links; reflowed and normalized list items with consistent hyphenation; adjusted headers and spacing; minor content reorganizations in Exp3 (added new project item and "Advice for Others" section); all changes purely presentational.
Fanatics Interview Experiences
Interview/Fanatics/Fanatics_Exp1/index.md, Interview/Fanatics/Fanatics_Exp2/index.md
Added practice problem links and practice blocks; reformatted section headers and bullet spacing; added "My key takeaways" subsection in Exp1; minor emphasis styling and punctuation normalization; no logic changes.
Goldman Sachs Interview Experiences
Interview/GoldmanSachs/Gs_Exp1/index.md, Interview/GoldmanSachs/Gs_Exp2/index.md, Interview/GoldmanSachs/Gs_Exp3/index.md, Interview/GoldmanSachs/Gs_Exp4/index.md
Formatting adjustments including note block style conversions, bulleted list reorganization, and emphasis changes; link corrections and standardization; minor heading and spacing refinements; no semantic changes to outcomes.
Google Interview Experience
Interview/Google/Google_Exp1/index.md
Expanded Round 3 DSA description with step-by-step discussion notes; added clarifications to Overall Experience section; reworked Advice for Others into explicit bullet points; added explicit Result section content; minor formatting adjustments.
LinkedIn Interview Experience
Interview/LinkedIn/LinkedIn_Exp1/index.md
Heading punctuation adjustments and italicization tweaks; updated link URLs; minor wording changes and formatting consistency; no structural or flow changes.
MathWorks Interview Experience
Interview/MathWorks/MathWork_Exp1/index.md
Reorganized and reflowed bullet points across sections; converted inline lists to explicit bulletization; updated HR questions to bulleted format; split compound items into separate bullets; minor punctuation and capitalization adjustments.
Microsoft Interview Experience
Interview/Microsoft/Microsoft_Exp1/index.md
Expanded Coding Questions with DP practice notes; reformed Computer Networks section into bulleted list; added descriptive text to Round 2 HR section with enumerated questions; inserted Microsoft values emphasis; updated external resource links; normalized bullet group formatting.
Optum Interview Experiences
Interview/Optum/Optum_Exp1/index.md, Interview/Optum/Optum_Exp2/index.md, Interview/Optum/Optum_Exp3/index.md, Interview/Optum/Optum_Exp4/index.md, Interview/Optum/Optum_Exp5/index.md, Interview/Optum/Optum_Exp6/index.md, Interview/Optum/Optum_Exp7/index.md, Interview/Optum/Optum_Exp8/index.md, Interview/Optum/Optum_Exp9/index.md, Interview/Optum/Optum_Exp10/index.md, Interview/Optum/Optum_Exp11/index.md
Consistent formatting and link updates across all 11 experiences; reflowed HR/Technical sections into explicit bullet lists; added prep/helpful resources subsections; updated GeeksforGeeks links; reorganized Related Links sections with new URL mappings; minor heading spacing and section re-emphasis.
Publicis Sapient Interview Experiences
Interview/Publicis_Sapient/Ps_Exp1/index.md, Interview/Publicis_Sapient/Ps_Exp2/index.md, Interview/Publicis_Sapient/Ps_Exp3/index.md, Interview/Publicis_Sapient/Ps_Exp4/index.md
Converted plain reference lines to markdown-formatted clickable links; adjusted whitespace and bold headings; expanded interview sections with new questions and entries; corrected/standardized link text; added "Advice for Others" section in Ps_Exp3; reformatted reference lists to consistent markdown style.
Stripe Interview Experience
Interview/Stripe/Stripe_Exp1/index.md
Added markdown styling (bold/italic) and blank lines; inserted note about language choice; revised Advice section; expanded reference list entries; purely formatting and copy edits.
Tech Mahindra Interview Experiences
Interview/Tech_Mahindra/Tm_Exp1/index.md, Interview/Tech_Mahindra/Tm_Exp2/index.md, Interview/Tech_Mahindra/Tm_Exp3/index.md
Converted reference sections from plain text to bullet lists with proper links; added explicit bullet items for coding platforms; expanded round details with bullet lists; standardized headings and removed formatting inconsistencies; updated shared-by line emphasis; no logic or content semantic changes.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Poem

🐰 With carrot-sized care, I've hopped through each link,
Converting grey URLs to markdown so slick.
Bullets aligned and spacing just right,
Interview docs now polished and bright!
No logic was changed, just presentation's delight, 🎉

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning The linked issue #114 reports visual clarity problems in diagram grey areas, but the PR changes are exclusively formatting, link styling, and markdown edits to interview markdown files with no diagram-related changes. The PR addresses broken links (the stated objective) but does not address the linked issue #114 about diagram visibility. Either update the PR to fix diagram visibility issues or unlink issue #114 and link the relevant issue instead.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title 'Fix : Broken links for interview section' clearly describes fixing broken links in the interview section, which aligns with the stated PR objective of correcting broken links.
Out of Scope Changes check ✅ Passed All changes are formatting, markdown styling, link conversions, and content reorganization of interview documentation files with no unrelated modifications detected.
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
  • Post copyable unit tests in a comment
  • Commit unit tests in branch link_fix

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

@RAJ8664 RAJ8664 merged commit 66822f4 into main Mar 7, 2026
7 of 8 checks passed
@RAJ8664 RAJ8664 deleted the link_fix branch March 7, 2026 22: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.

In grey part of some diagrams , the content is not clearly visible !

1 participant