Skip to content

Conversation

shaobo-he-aws
Copy link
Contributor

@shaobo-he-aws shaobo-he-aws commented Sep 23, 2025

Description of changes

Issue #, if available

Checklist for requesting a review

The change in this PR is (choose one, and delete the other options):

  • A change "invisible" to users (e.g., documentation, changes to "internal" crates like cedar-policy-core, cedar-validator, etc.)

I confirm that this PR (choose one, and delete the other options):

  • Does not update the CHANGELOG because my change does not significantly impact released code.

I confirm that cedar-spec (choose one, and delete the other options):

  • Does not require updates because my change does not impact the Cedar formal model or DRT infrastructure.

I confirm that docs.cedarpolicy.com (choose one, and delete the other options):

  • Does not require updates because my change does not impact the Cedar language specification.

Copy link

Coverage Report

Head Commit: a9d8c0f827bd2205daa7b95f6207ce282fd4623c

Base Commit: 1b3275b696ab10a3a8351df8cbfe353c99df4cda

Download the full coverage report.

Coverage of Added or Modified Lines of Rust Code

Required coverage: 80.00%

Actual coverage: 91.67%

Status: PASSED ✅

Details
File Status Covered Coverage Missed Lines
cedar-policy-core/src/tpe/entities.rs 🟢 11/12 91.67% 531

Coverage of All Lines of Rust Code

Required coverage: 80.00%

Actual coverage: 84.07%

Status: PASSED ✅

Details
Package Status Covered Coverage Base Coverage
cedar-language-server 🟢 4619/5120 90.21% --
cedar-policy 🟡 3513/4929 71.27% --
cedar-policy-cli 🔴 675/1054 64.04% --
cedar-policy-core 🟢 21450/25212 85.08% --
cedar-policy-formatter 🟢 930/1112 83.63% --
cedar-policy-symcc 🟢 4715/5249 89.83% --
cedar-wasm 🔴 0/28 0.00% --

@shaobo-he-aws shaobo-he-aws merged commit d0e0bbb into main Sep 23, 2025
22 checks passed
@shaobo-he-aws shaobo-he-aws deleted the actions-in-schema-tpe branch September 23, 2025 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants