Updated Indian Ambassador Evangelists (Business Incubation Centre IKGPTU) #14
Conversation
📝 WalkthroughWalkthroughThe India row in Changes
Sequence Diagram(s)(omitted — change is documentation-only and does not introduce multi-component control flow) Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@Evangelists.md`:
- Line 35: The table row for "India | IKGPTU, Punjab | Business Incubation
Centre, IKGPTU | Manish Tiwari | `@manishtiwari_25546` | `@predictivemanish`
|TODO" has one extra cell ("TODO") causing a 7th column; remove the trailing
"TODO" cell so the row contains exactly six pipe-separated cells to match the
table header, e.g., delete the final " |TODO" from that row (locate the row by
the unique text "Manish Tiwari" or "@manishtiwari_25546").
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@Evangelists.md`:
- Line 35: Trim the leading spaces from the Discord username cell in the
markdown table row containing "India | IKGPTU, Punjab | Business Incubation
Centre, IKGPTU | Manish Tiwari | `@manishtiwari_25546` | `@predictivemanish` |" by
replacing the value " `@predictivemanish`" with "@predictivemanish" so the
Discord username column has no leading whitespace and matches the table
formatting.
♻️ Duplicate comments (1)
Evangelists.md (1)
35-35: The table structure issue from the previous review remains unresolved.This row still has 7 columns instead of 6. The extra trailing empty cell must be removed to match the table header structure and fix the markdown rendering issue.
🔧 Proposed fix to remove the extra column
-| India | IKGPTU, Punjab | Business Incubation Centre, IKGPTU | Manish Tiwari | `@manishtiwari_25546` | `@predictivemanish` | | +| India | IKGPTU, Punjab | Business Incubation Centre, IKGPTU | Manish Tiwari | `@manishtiwari_25546` | `@predictivemanish` |Based on static analysis hints from markdownlint-cli2.
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@Evangelists.md`:
- Line 35: The table row in Evangelists.md contains an extra trailing empty cell
making 7 columns while the header defines 6; edit the row that currently reads
"| India | IKGPTU, Punjab | Business Incubation Centre,
IKGPTU | Manish Tiwari | `@manishtiwari_25546` |
`@predictivemanish` | |" to remove the final empty cell (the
trailing "| |") so the row has exactly six cells matching the header.
🧹 Nitpick comments (1)
Evangelists.md (1)
35-35: Consider adding backticks to usernames for inline code formatting.Username handles are often formatted with backticks in markdown tables to provide visual distinction and inline code formatting.
♻️ Optional formatting improvement
-| India | IKGPTU, Punjab | Business Incubation Centre, IKGPTU | Manish Tiwari | `@manishtiwari_25546` | `@predictivemanish` | +| India | IKGPTU, Punjab | Business Incubation Centre, IKGPTU | Manish Tiwari | `@manishtiwari_25546` | `@predictivemanish` |
| | Germany | -- | TODO | TODO | TODO | TODO | | ||
| | Egypt | -- | TODO | TODO | TODO | TODO | | ||
| | India | -- | TODO | TODO | TODO | TODO | | ||
| | India | IKGPTU, Punjab | Business Incubation Centre, IKGPTU | Manish Tiwari | @manishtiwari_25546 | @predictivemanish | | |
There was a problem hiding this comment.
Fix table structure: remove the extra trailing column.
The table row has 7 cells but the header defines only 6 columns. The extra empty cell at the end breaks the markdown table structure and triggers a linting error.
🔧 Proposed fix to remove the extra column
-| India | IKGPTU, Punjab | Business Incubation Centre, IKGPTU | Manish Tiwari | `@manishtiwari_25546` | `@predictivemanish` | |
+| India | IKGPTU, Punjab | Business Incubation Centre, IKGPTU | Manish Tiwari | `@manishtiwari_25546` | `@predictivemanish` |Based on static analysis hints from markdownlint-cli2.
🧰 Tools
🪛 markdownlint-cli2 (0.18.1)
35-35: Table column count
Expected: 6; Actual: 7; Too many cells, extra data will be missing
(MD056, table-column-count)
🤖 Prompt for AI Agents
In `@Evangelists.md` at line 35, The table row in Evangelists.md contains an extra
trailing empty cell making 7 columns while the header defines 6; edit the row
that currently reads "| India | IKGPTU, Punjab | Business
Incubation Centre, IKGPTU | Manish Tiwari | `@manishtiwari_25546`
| `@predictivemanish` | |" to remove the final empty cell
(the trailing "| |") so the row has exactly six cells matching the
header.
|
Hey @PredictiveManish , for Evangelists we plan to have one per college, not one per country. Reopening the PR if you are still interested. |
Ok, yes I thought it's for country basis. Yes I'm interested thanks for reopening PR. |
Great, welcome to AOSSIE. Could you please answer the following questions: What student community are you a part of (With links to verifiable sources) ? |
2 participants
Through this PR, added Indian ambassador for Evangelists, It's my pleasure to get a chance for this work. I'll help this organization to be famous in our university IK Gujral Punjab Technical University, and it's around 200 affiliated colleges. As our university has direct affiliation with around 200+ colleges here in India itself.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.