docs: add Star History chart image with sealed token - #66
Conversation
Co-Authored-By: Afonso Dutra Nogueira Filho <afonsoft@gmail.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
There was a problem hiding this comment.
🔍 Duplicate README files may conflict on case-insensitive file systems
The repository has both README.md and readme.md at the root with different content. On case-insensitive file systems (Windows, macOS default), these two files would collide — only one would be visible, and git operations could behave unpredictably. This is a pre-existing issue, but since this PR modifies both files and the project explicitly claims cross-platform support (Windows, Linux, macOS per AGENTS.md:103), it's worth noting. GitHub renders README.md by default, so readme.md may not be serving its intended purpose.
Was this helpful? React with 👍 or 👎 to provide feedback.
|
|
||
| See the [star history chart](https://www.star-history.com/?repos=afonsoft%2FQRCoder.Core&type=date&legend=top-left) for this repository.<br/> | ||
| Veja o [gráfico de histórico de estrelas](https://www.star-history.com/?repos=afonsoft%2FQRCoder.Core&type=date&legend=top-left) deste repositório. | ||
| [](https://www.star-history.com/?repos=afonsoft%2FQRCoder.Core&type=date&legend=top-left) |
There was a problem hiding this comment.
🟡 Portuguese description removed from the bilingual README's star history section
The Portuguese translation of the star history description is dropped (readme.md:287) when replacing the two bilingual lines with a single English-only image badge, so the bilingual README loses its Portuguese content in that section.
Impact: The bilingual README no longer provides Portuguese text for the star history section, breaking the project's bilingual documentation standard.
Bilingual documentation rule violation details
The readme.md file is the project's bilingual (EN/PT-BR) README — all section headers use the dual-language pattern (e.g., "Star History / Histórico de Estrelas", "License / Licença").
Before this change, the Star History section had two lines:
- English:
See the [star history chart](...) for this repository. - Portuguese:
Veja o [gráfico de histórico de estrelas](...) deste repositório.
After this change, both lines are replaced with a single image badge with English-only alt text (Star History Chart). The Portuguese description is entirely removed.
AGENTS.md explicitly requires under "DO (Faça)": "Documentação bilíngue (pt-BR + en-US)"
| [](https://www.star-history.com/?repos=afonsoft%2FQRCoder.Core&type=date&legend=top-left) | |
| [](https://www.star-history.com/?repos=afonsoft%2FQRCoder.Core&type=date&legend=top-left) | |
| See the star history chart for this repository. / Veja o gráfico de histórico de estrelas deste repositório. |
Was this helpful? React with 👍 or 👎 to provide feedback.
| ## Star History | ||
|
|
||
| See the [star history chart](https://www.star-history.com/?repos=afonsoft%2FQRCoder.Core&type=date&legend=top-left) for this repository. | ||
| [](https://www.star-history.com/?repos=afonsoft%2FQRCoder.Core&type=date&legend=top-left) |
There was a problem hiding this comment.
📝 Info: Star-history sealed_token in URL is a public embedding token, not a secret
The sealed_token parameter in the star-history API URL (e.g., https://api.star-history.com/chart?...&sealed_token=a1KQPz...) might appear to be a committed secret. However, star-history.com provides these tokens publicly for anyone embedding their chart badges — they are not private API keys or credentials. This does not violate the AGENTS.md Hard Rule H4 ("Secrets: nunca commitar .env, tokens ou API keys") since it is not a secret credential.
Was this helpful? React with 👍 or 👎 to provide feedback.
Qodana Community for .NET1272 new problems were found
💡 Qodana analysis was run in the pull request mode: only the changed files were checked View the detailed Qodana reportTo be able to view the detailed Qodana report, you can either:
To get - name: 'Qodana Scan'
uses: JetBrains/qodana-action@v2026.1.3
with:
upload-result: trueContact Qodana teamContact us at qodana-support@jetbrains.com
|
1 participant
All Submissions:
Changes to Core Features:
Description
Updates the Star History section in the README files to display the live chart image using the sealed token for
afonsoft/QRCoder.Core.Changes
README.md,readme.md, andREADME.pt-br.md.https://www.star-history.com/?repos=afonsoft%2FQRCoder.Core&type=date&legend=top-left.https://api.star-history.com/chart?repos=afonsoft/QRCoder.Core&type=date&legend=top-left&sealed_token=....Verification
README.mdandREADME.pt-br.mdrendered correctly with the new image markdown.readme.md(bilingual) updated with the same image.Impact
Pure documentation change; no source code or tests affected.
Link to Devin session: https://app.devin.ai/sessions/f862be50b6ad41bfa3bb93c39e8a89a6
Requested by: @afonsoft