Add Contributors and fix broken markdown in README_pt_br#8375
Conversation
Remove broken markdown under '## Apoiadores', and add '## Contribuidores'
BraianS
requested review from
bijin-bruno,
helloanoop,
lohit-bruno,
naman-bruno,
sid-bruno,
utkarsh-bruno and
vijayh-bruno
as code owners
WalkthroughThe Portuguese README replaces the sponsors section with a collaborators section that links to the repository contributor graph and shows the contrib.rocks contributor image. ChangesPortuguese README update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 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 |
![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
🧹 Nitpick comments (1)
docs/readme/readme_pt_br.md (1)
96-96: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low valueAdd alt text to the contrib.rocks image for accessibility.
The
<img>tag is missing analtattribute. While this image is decorative, explicit empty alt (alt="") or descriptive text improves accessibility compliance.🛠️ Proposed fix
- <img src="https://contrib.rocks/image?repo=usebruno/bruno" /> + <img src="https://contrib.rocks/image?repo=usebruno/bruno" alt="Contribuidores do Bruno" />🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@docs/readme/readme_pt_br.md` at line 96, The README image tag in the contrib.rocks section is missing accessibility text; update the `<img>` element to include an `alt` attribute, using either an empty value for decorative content or concise descriptive text. Locate the image markup in the README snippet and adjust the existing image tag accordingly so it complies with accessibility requirements.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/readme/readme_pt_br.md`:
- Around line 92-98: The README has duplicate contributor sections: the newly
added `### Colaboradores` block duplicates the existing contributor block
already present later in the file. Remove the earlier inserted section and keep
the existing bottom placement that matches the other localized READMEs, or if
the intent is a terminology change, rename the existing `### Contribuidores`
heading instead of adding a second block. Also delete the `### Apoiadores`
section only if it is no longer needed, using the existing contributor section
as the single source of truth.
---
Nitpick comments:
In `@docs/readme/readme_pt_br.md`:
- Line 96: The README image tag in the contrib.rocks section is missing
accessibility text; update the `<img>` element to include an `alt` attribute,
using either an empty value for decorative content or concise descriptive text.
Locate the image markup in the README snippet and adjust the existing image tag
accordingly so it complies with accessibility requirements.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 468adda0-8cb9-4d3b-8e42-39a87c5e045d
📒 Files selected for processing (1)
docs/readme/readme_pt_br.md
| ### Colaboradores | ||
|
|
||
| #### Apoiadores Gold | ||
|
|
||
| <img src="../../assets/images/sponsors/samagata.png" width="150"/> | ||
|
|
||
| #### Apoiadores Silver | ||
|
|
||
| <img src="../../assets/images/sponsors/commit-company.png" width="70"/> | ||
|
|
||
| #### Apoiadores Bronze | ||
|
|
||
| <a href="https://zuplo.link/bruno"> | ||
| <img src="../../assets/images/sponsors/zuplo.png" width="120"/> | ||
| </a> | ||
| <div align="center"> | ||
| <a href="https://github.com/usebruno/bruno/graphs/contributors"> | ||
| <img src="https://contrib.rocks/image?repo=usebruno/bruno" /> | ||
| </a> | ||
| </div> |
There was a problem hiding this comment.
🎯 Functional Correctness | 🔴 Critical | ⚡ Quick win
Duplicate contributor sections - file now contains two identical contributor blocks.
The new ### Colaboradores section (lines 92-98) duplicates the existing ### Contribuidores section at lines 152-158, which already has the same markup. Based on cross-file context, other language READMEs (French, English) place this section near the bottom before the license. The existing placement at 152-158 follows this pattern; the new addition at 92-98 does not.
Remove the newly added 92-98 block and simply delete the ### Apoiadores section without replacement, since ### Contribuidores already exists in the correct location. If you intend to rename "Contribuidores" to "Colaboradores" for consistency, replace the heading at line 152 instead.
🧰 Tools
🪛 markdownlint-cli2 (0.22.1)
[warning] 96-96: Images should have alternate text (alt text)
(MD045, no-alt-text)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@docs/readme/readme_pt_br.md` around lines 92 - 98, The README has duplicate
contributor sections: the newly added `### Colaboradores` block duplicates the
existing contributor block already present later in the file. Remove the earlier
inserted section and keep the existing bottom placement that matches the other
localized READMEs, or if the intent is a terminology change, rename the existing
`### Contribuidores` heading instead of adding a second block. Also delete the
`### Apoiadores` section only if it is no longer needed, using the existing
contributor section as the single source of truth.
1 participant
Description
Remove broken markdown under
## Apoiadores, and add## Contribuidores.Contribution Checklist:
Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.
Publishing to New Package Managers
Please see here for more information.
Summary by CodeRabbit