Skip to content

Add usages for the constraints.#483

Merged
Blacksmoke16 merged 35 commits intoathena-framework:masterfrom
tamdaz:master
Dec 14, 2024
Merged

Add usages for the constraints.#483
Blacksmoke16 merged 35 commits intoathena-framework:masterfrom
tamdaz:master

Conversation

@tamdaz
Copy link
Contributor

@tamdaz tamdaz commented Dec 13, 2024

Context

Reference to issue #201

This pull request consists to add usages that provide a better understanding of their representations, so, developers can easily use the constraints.

Changelog

  • Add example usages to AVD::Constraints docs

Before merging, remember to add the athena-framework/athena prefix to the PR number in the PR title

@codecov
Copy link

codecov bot commented Dec 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

Flag Coverage Δ
compiled 8.14% <ø> (ø)
unit 92.75% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
clock 98.18% <ø> (ø)
console 92.61% <ø> (ø)
dependency_injection 64.28% <ø> (ø)
dotenv 96.80% <ø> (ø)
event_dispatcher 82.75% <ø> (ø)
framework 94.16% <ø> (ø)
image_size 95.03% <ø> (ø)
mercure 93.37% <ø> (ø)
negotiation 97.69% <ø> (ø)
routing 92.96% <ø> (ø)
serializer 87.23% <ø> (ø)
spec 93.54% <ø> (ø)
validator 92.37% <ø> (ø)

📢 Thoughts on this report? Let us know!

Copy link
Member

@Blacksmoke16 Blacksmoke16 left a comment

Choose a reason for hiding this comment

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

Thanks for this. Overall it looks good to me, but left a couple comments.

@tamdaz tamdaz requested a review from Blacksmoke16 December 14, 2024 18:09
@Blacksmoke16 Blacksmoke16 enabled auto-merge (squash) December 14, 2024 18:21
@Blacksmoke16 Blacksmoke16 merged commit 3bcaa0b into athena-framework:master Dec 14, 2024
PallasAthenaie pushed a commit to athena-framework/validator that referenced this pull request Dec 14, 2024
@tamdaz
Copy link
Contributor Author

tamdaz commented Dec 14, 2024

Perfect, thank you. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

Add examples to validation constraint docs

2 participants