Partially fix broken reference links in interview experiences#106
Partially fix broken reference links in interview experiences#106RAJ8664 merged 2 commits intoRAJ8664:mainfrom
Conversation
|
@KanikaPunia119 is attempting to deploy a commit to the RAJ ROY's projects Team on Vercel. A member of the Team first needs to authorize it. |
📝 WalkthroughWalkthroughThe PR updates multiple interview preparation documents by replacing external reference links with alternative sources (dev.to articles, updated GeeksforGeeks URLs), and revises internship program titles and years across Stripe and Tech Mahindra experience files. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
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. Comment |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
📜 Review details
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (6)
Interview/Publicis_Sapient/Ps_Exp3/index.mdInterview/Stripe/Stripe_Exp1/index.mdInterview/Stripe/index.mdInterview/Tech_Mahindra/Tm_Exp1/index.mdInterview/Tech_Mahindra/Tm_Exp3/index.mdInterview/Tech_Mahindra/index.md
🧰 Additional context used
🪛 markdownlint-cli2 (0.18.1)
Interview/Tech_Mahindra/Tm_Exp3/index.md
83-83: Bare URL used
(MD034, no-bare-urls)
101-101: Bare URL used
(MD034, no-bare-urls)
147-147: Bare URL used
(MD034, no-bare-urls)
148-148: Bare URL used
(MD034, no-bare-urls)
149-149: Bare URL used
(MD034, no-bare-urls)
⏰ 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)
- GitHub Check: Build Docs
🔇 Additional comments (6)
Interview/Publicis_Sapient/Ps_Exp3/index.md (1)
72-72: LGTM - consistent reference link update.The GeeksforGeeks OS introduction set-1 URL is valid, and the changes are consistent across both occurrences.
Also applies to: 153-153
Interview/Stripe/index.md (1)
3-3: LGTM - metadata clarity improvements.Heading and title updates add specificity (Winter, 2026 year) to the internship listing.
Also applies to: 9-9
Interview/Tech_Mahindra/Tm_Exp1/index.md (1)
126-126: LGTM - minor punctuation improvement.Grammar/readability enhancement with no functional impact.
Interview/Stripe/Stripe_Exp1/index.md (1)
20-20: ✅ Reference links are valid and accessible.GeeksforGeeks C++ regex page, dev.to clean code article, and dev.to JSON parser article are all confirmed accessible and relevant to the interview content.
Also applies to: 59-59, 108-108
Interview/Tech_Mahindra/Tm_Exp3/index.md (1)
83-83: ✅ Reference links are updated and relevant.Updated reference links point to more targeted content on GeeksforGeeks.
Also applies to: 101-101
Interview/Tech_Mahindra/index.md (1)
3-3: Verify internship year consistency across PR.This file updates internship years to 2025, but Interview/Stripe/index.md uses 2026 for its internship. Confirm whether this year discrepancy is intentional or should be aligned across all experience files.
Also applies to: 9-11
| - https://www.geeksforgeeks.org/cpp/regex-regular-expression-in-c/ | ||
| - https://dev.to/favourmark05/writing-clean-code-best-practices-and-principles-3amh | ||
| - https://google.github.io/styleguide/ | ||
| - https://www.geeksforgeeks.org/rest-api-introduction/ | ||
| - https://curl.se/libcurl/ | ||
| - https://www.geeksforgeeks.org/json-parsing-in-c/ | ||
| - https://dev.to/uponthesky/c-making-a-simple-json-parser-from-scratch-250g |
There was a problem hiding this comment.
Format bare URLs in references section to comply with markdownlint (MD034).
The reference links section contains bare URLs that violate markdownlint's MD034 rule (no-bare-urls). Wrap them in markdown link syntax for consistency:
## 🔗 Reference Links
- https://stripe.com/jobs
+ - [Stripe Jobs](https://stripe.com/jobs)
- https://www.geeksforgeeks.org/cpp/regex-regular-expression-in-c/
+ - [Regular Expressions in C++ – GeeksforGeeks](https://www.geeksforgeeks.org/cpp/regex-regular-expression-in-c/)
- https://dev.to/favourmark05/writing-clean-code-best-practices-and-principles-3amh
+ - [Writing Clean Code: Best Practices – DEV](https://dev.to/favourmark05/writing-clean-code-best-practices-and-principles-3amh)
- https://google.github.io/styleguide/
+ - [Google Style Guide](https://google.github.io/styleguide/)
- https://www.geeksforgeeks.org/rest-api-introduction/
+ - [REST API Introduction – GeeksforGeeks](https://www.geeksforgeeks.org/rest-api-introduction/)
- https://curl.se/libcurl/
+ - [libcurl Documentation](https://curl.se/libcurl/)
- https://dev.to/uponthesky/c-making-a-simple-json-parser-from-scratch-250g
+ - [JSON Parser in C++ – DEV](https://dev.to/uponthesky/c-making-a-simple-json-parser-from-scratch-250g)Committable suggestion skipped: line range outside the PR's diff.
🤖 Prompt for AI Agents
In Interview/Stripe/Stripe_Exp1/index.md around lines 144 to 149 the references
list contains bare URLs which violate markdownlint rule MD034; replace each bare
URL with markdown link syntax [title](url) — choose an appropriate short title
for each reference (e.g., "GeeksforGeeks: C++ regex", "Dev.to: Writing Clean
Code", "Google Style Guide", "GeeksforGeeks: REST API", "curl: libcurl",
"Dev.to: Simple JSON Parser") so every entry uses [text](https://...) format and
the file no longer contains bare URLs.
| # Inside stories. Real experiences. Tech Mahindra interviews, unfiltered. | ||
|
|
||
| ## 📚 SDE | ||
| ## 📚 SDE Intern (TechnoRise Intern) |
There was a problem hiding this comment.
Fix table row numbering error in row 3.
Line 11 has row index | 2 | which duplicates the row above. This should be | 3 | to correctly index the third row:
🔎 Proposed fix for row numbering
| # | Role | View |
|:-:|:---------------------------------|:--------------------------------------------------------------------------------------------:|
| 1 | **TechnoRise Intern 2025** | [📥 Open](./Tm_Exp1) |
| 2 | **TechnoRise Intern 2025** | [📥 Open](./Tm_Exp2) |
-| 2 | **TechnoRise Intern 2025** | [📥 Open](./Tm_Exp3) |
+| 3 | **TechnoRise Intern 2025** | [📥 Open](./Tm_Exp3) |Also applies to: 9-11
🤖 Prompt for AI Agents
In Interview/Tech_Mahindra/index.md around line 3 (and specifically table rows
covering lines 9-11), the markdown table duplicates the row index "| 2 |" for
the third entry; update that cell to "| 3 |" (and verify subsequent row indexes
in the table are incremented correctly) so row numbering is sequential and
unique.
| - https://www.geeksforgeeks.org/operating-systems/ | ||
| - https://www.geeksforgeeks.org/dbms/ | ||
| - https://www.geeksforgeeks.org/reactjs-tutorial/ | ||
| - https://www.geeksforgeeks.org/reactjs/react/ |
There was a problem hiding this comment.
Format bare URLs in references section to comply with markdownlint (MD034).
Similar to Stripe_Exp1/index.md, wrap bare URLs in markdown link syntax:
## 🔗 Reference Links
- https://www.techmahindra.com/
+ - [Tech Mahindra](https://www.techmahindra.com/)
- https://www.geeksforgeeks.org/data-structures/
+ - [Data Structures – GeeksforGeeks](https://www.geeksforgeeks.org/data-structures/)
- https://leetcode.com/problemset/all/
+ - [LeetCode Problems](https://leetcode.com/problemset/all/)
- https://www.indiabix.com/aptitude/questions-and-answers/
+ - [Aptitude Questions – IndiaBIX](https://www.indiabix.com/aptitude/questions-and-answers/)
- https://www.indiabix.com/verbal-ability/questions-and-answers/
+ - [Verbal Ability – IndiaBIX](https://www.indiabix.com/verbal-ability/questions-and-answers/)
- https://www.geeksforgeeks.org/operating-systems/
+ - [Operating Systems – GeeksforGeeks](https://www.geeksforgeeks.org/operating-systems/)
- https://www.geeksforgeeks.org/dbms/
+ - [DBMS – GeeksforGeeks](https://www.geeksforgeeks.org/dbms/)
- https://www.geeksforgeeks.org/reactjs/react/
+ - [React.js – GeeksforGeeks](https://www.geeksforgeeks.org/reactjs/react/)
- https://www.geeksforgeeks.org/javascript-tutorial/
+ - [JavaScript Tutorial – GeeksforGeeks](https://www.geeksforgeeks.org/javascript-tutorial/)
- https://www.geeksforgeeks.org/hr/hr-interview-questions/
+ - [HR Interview Questions – GeeksforGeeks](https://www.geeksforgeeks.org/hr/hr-interview-questions/)Also applies to: 149-149
🧰 Tools
🪛 markdownlint-cli2 (0.18.1)
147-147: Bare URL used
(MD034, no-bare-urls)
🤖 Prompt for AI Agents
In Interview/Tech_Mahindra/Tm_Exp3/index.md around lines 147 and 149, the
references section contains bare URLs which violate markdownlint rule MD034;
replace each bare URL with markdown link syntax by providing a short descriptive
link text followed by the URL in parentheses (e.g. [GeeksforGeeks React
Guide](https://www.geeksforgeeks.org/reactjs/react/)), ensuring both occurrences
are updated and the document remains valid Markdown.
2 participants
👋 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 checkboxSummary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.