Conversation
…, Postman collection, TypeScript, and Prettier
There was a problem hiding this comment.
Pull request overview
Updates repository/package presentation by adding additional README badges and recording the change via Changesets, while also adjusting commitlint’s maximum header length.
Changes:
- Add promotional/technical badges to
README.md(API key, docs, Postman, TypeScript, Prettier). - Add a patch changeset describing the README badge update.
- Increase commitlint
header-max-lengthfrom 100 to 255.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
commitlint.config.js |
Loosens commit message header length limit (100 → 255). |
README.md |
Adds new badges for API key/docs/Postman/TypeScript/Prettier. |
.changeset/add-readme-badges.md |
Records the README badge update as a patch release entry. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| [](https://dashboard.astrology-api.io/) | ||
| [](https://api.astrology-api.io/rapidoc) | ||
| [](https://api.astrology-api.io/best-astrology-api-postman.json) |
There was a problem hiding this comment.
Badge styling is currently mixed: the new badges use style=for-the-badge while the existing CI/npm/license badges use the default shields style. Consider using a consistent shields style across all badges (either remove style=for-the-badge from the new ones or apply it to the existing ones) to avoid inconsistent sizing/visual layout.
| [](https://dashboard.astrology-api.io/) | |
| [](https://api.astrology-api.io/rapidoc) | |
| [](https://api.astrology-api.io/best-astrology-api-postman.json) | |
| [](https://dashboard.astrology-api.io/) | |
| [](https://api.astrology-api.io/rapidoc) | |
| [](https://api.astrology-api.io/best-astrology-api-postman.json) |
| ], | ||
| 'subject-case': [2, 'never', ['start-case', 'pascal-case', 'upper-case']], | ||
| 'header-max-length': [2, 'always', 100], | ||
| 'header-max-length': [2, 'always', 255], |
There was a problem hiding this comment.
The PR title indicates this is a docs/README-only change, but this modifies commitlint behavior by increasing header-max-length from 100 to 255. If this change is intentional, it should be called out in the PR description (with rationale); otherwise it should be reverted to avoid unexpectedly loosening commit message validation for the repo.
| [](https://www.npmjs.com/package/@astro-api/astroapi-typescript) | ||
| [](https://www.npmjs.com/package/@astro-api/astroapi-typescript) | ||
| [](https://opensource.org/licenses/MIT) | ||
| []() |
There was a problem hiding this comment.
This TypeScript badge uses an empty link target (), which renders as a non-functional link. Please either provide a valid URL (e.g., TypeScript website/releases or repo) or remove the link wrapper if it’s meant to be non-clickable.
| []() | |
| [](https://www.typescriptlang.org/) |
1 participant
No description provided.