Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

chore: improve and extend README content #157

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
J3m5 merged 5 commits into main from chore/readme
Jul 14, 2025
Merged

chore: improve and extend README content #157

J3m5 merged 5 commits into main from chore/readme
Jul 14, 2025

Conversation

@J3m5
Copy link
Contributor

@J3m5 J3m5 commented Jul 3, 2025
edited
Loading

  • πŸ“ Enhanced the README with additional content and sections for better clarity.

  • πŸ› Fixed broken links to ensure all references are valid.

  • 🎨 Reformatted the document for improved readability.

  • πŸ–ΌοΈ Added a custom logo

Check the rendered changes
image

slax57 reacted with heart emoji
@J3m5 J3m5 requested a review from Copilot July 3, 2025 16:06
Copy link

github-actions bot commented Jul 3, 2025
edited
Loading

Coverage Report

Status Category Percentage Covered / Total
🟒 Lines 71.35% (🎯 70%) 289 / 405
🟒 Statements 71.39% (🎯 70%) 292 / 409
🟒 Functions 71.64% (🎯 70%) 48 / 67
🟒 Branches 59.63% (🎯 58%) 229 / 384
File CoverageNo changed files found.
Generated in workflow #252 for commit 07f2ed3 by the Vitest Coverage Report Action

@J3m5 J3m5 requested review from alanpoulain and dunglas July 3, 2025 16:06
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR overhauls the README to improve clarity, navigation, and completeness by reorganizing the header, adding new sections, and fixing links.

  • Introduce a centered title banner with updated badges and quick links.
  • Add "Key Features," expanded installation options (Yarn, Pnpm, Bun), usage examples, and detailed TypeScript definitions.
  • Reformat the OpenAPI support details into a rules table and expand sections on contributing, contributors, star history, credits, and license.

Copy link
Member

dunglas commented Jul 3, 2025

Please remove the HTML. It's useless here and make the file less readable.

J3m5 reacted with thumbs up emoji

Copy link
Member

dunglas commented Jul 3, 2025

The TOC isn't useful either, it is generated automatically by GitHub and our website.

J3m5 reacted with thumbs up emoji

Copy link
Contributor Author

J3m5 commented Jul 4, 2025
edited
Loading

The TOC isn't useful either, it is generated automatically by GitHub and our website.

While the TOC is automatically generated by GitHub, it serves as a quick-access navigation shortcut to the most relevant sections.

Copy link
Contributor Author

J3m5 commented Jul 4, 2025

Please remove the HTML. It's useless here and make the file less readable.

It just doesn’t look as nice to me
image
image

Copy link
Contributor Author

J3m5 commented Jul 4, 2025
edited
Loading

I've applied your suggestions, @dunglas.
The only HTML left is for the chart history image, since it allows the chart to adapt to dark or light mode based on the user's preference.

@J3m5 J3m5 requested a review from a team July 7, 2025 14:39
Copy link
Contributor Author

J3m5 commented Jul 13, 2025

@api-platform/core-team-js

Quick ping: since the last review I’ve made some small changes (usage examples with await + destructured returns, reordered the install section, fixed links, and added a logo).

If there’s anything else you’d like changed before I merge, just let me know.

@J3m5 J3m5 merged commit d718a1f into main Jul 14, 2025
1 check passed
@J3m5 J3m5 deleted the chore/readme branch July 14, 2025 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@dunglas dunglas Awaiting requested review from dunglas

@alanpoulain alanpoulain Awaiting requested review from alanpoulain

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

AltStyle γ«γ‚ˆγ£γ¦ε€‰ζ›γ•γ‚ŒγŸγƒšγƒΌγ‚Έ (->γ‚ͺγƒͺγ‚ΈγƒŠγƒ«) /