Skip to content

[JsonPath] Add test documentation for the component #21090

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

Conversation

vinceAmstoutz
Copy link
Contributor

Updates the documentation to document JsonPathAssertionsTrait and related constraints (closes #20977).

Complete the previous work done in #21078.

@vinceAmstoutz vinceAmstoutz changed the title docs(json-path): add testing part doc for the component [JsonPath] Add test documentation for the component Jun 17, 2025
@vinceAmstoutz vinceAmstoutz force-pushed the docs/json-path/add-assertions-and-trait-the-component-doc branch from e4e14e1 to b1c2340 Compare June 17, 2025 08:51
@vinceAmstoutz vinceAmstoutz requested a review from OskarStark June 17, 2025 09:32
Copy link
Member

@javiereguiluz javiereguiluz left a comment

Choose a reason for hiding this comment

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

Nice work Vincent! Thanks.

@carsonbot carsonbot changed the title [JsonPath] Add test documentation for the component Add test documentation for the component Jun 18, 2025
@carsonbot carsonbot changed the title Add test documentation for the component [JsonPath] Add test documentation for the component Jun 18, 2025
@vinceAmstoutz
Copy link
Contributor Author

@alamirault a composer require phpunit/phpunit --dev is it necessary to correct this CI work? I can make a PR if necessary.

@vinceAmstoutz vinceAmstoutz force-pushed the docs/json-path/add-assertions-and-trait-the-component-doc branch from 85bc74a to 8d20ebb Compare June 19, 2025 09:26
@OskarStark
Copy link
Contributor

@alamirault a composer require phpunit/phpunit --dev is it necessary to correct this CI work? I can make a PR if necessary.

Might be a good idea, @symfony/docs WDYT?

@vinceAmstoutz vinceAmstoutz force-pushed the docs/json-path/add-assertions-and-trait-the-component-doc branch from 8d20ebb to f03d8e2 Compare June 19, 2025 10:05
@vinceAmstoutz
Copy link
Contributor Author

Rebased from 7.3 ✅

Copy link
Member

@alexandre-daubois alexandre-daubois left a comment

Choose a reason for hiding this comment

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

Try to wrap your lines at 80 chars overall 🙂

@OskarStark
Copy link
Contributor

~80 chars 😄 not a hard limit, but an orientation

@vinceAmstoutz
Copy link
Contributor Author

@alexandre-daubois Done for 80 👍

@vinceAmstoutz vinceAmstoutz force-pushed the docs/json-path/add-assertions-and-trait-the-component-doc branch from ad3f520 to b5c9c0a Compare June 19, 2025 13:29
@vinceAmstoutz
Copy link
Contributor Author

vinceAmstoutz commented Jun 19, 2025

But for lisst when I rendenring them it's look like :/ @alexandre-daubois @OskarStark
image

@alexandre-daubois
Copy link
Member

Try indenting with the rest, i.e. two spaces for each line under the list item

@vinceAmstoutz vinceAmstoutz force-pushed the docs/json-path/add-assertions-and-trait-the-component-doc branch from b5c9c0a to 24429d5 Compare June 19, 2025 13:57
@vinceAmstoutz
Copy link
Contributor Author

Try indenting with the rest, i.e. two spaces for each line under the list item

Done everywhere, its works as expected! 👍

@javiereguiluz javiereguiluz force-pushed the docs/json-path/add-assertions-and-trait-the-component-doc branch from 24429d5 to 87ccbb5 Compare June 19, 2025 14:21
@javiereguiluz javiereguiluz merged commit 481cb7a into symfony:7.3 Jun 19, 2025
2 of 3 checks passed
@javiereguiluz
Copy link
Member

Merged! Thanks a lot Vincent for your recent contributions to Symfony Docs!

@vinceAmstoutz vinceAmstoutz deleted the docs/json-path/add-assertions-and-trait-the-component-doc branch June 19, 2025 21:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[JsonPath] Add JsonPathAssertionsTrait and related constraints
5 participants