Skip to content

New Features from 0.25.2 release of cel-spec#174

Open
slott56 wants to merge 2 commits into
mainfrom
issue_172
Open

New Features from 0.25.2 release of cel-spec#174
slott56 wants to merge 2 commits into
mainfrom
issue_172

Conversation

@slott56
Copy link
Copy Markdown
Collaborator

@slott56 slott56 commented May 19, 2026

This has a new set of .textproto files, and revised .feature files.

The tags.toml is updated to flag new cel-spec test cases as "work-in-process" with the @wip tag on the tests.

The objective is to have a clean separation of features that may (or may not) pass acceptance tests. See https://github.com/cloud-custodian/cel-python/discussions for some discussion topics on how to address the WIP tags.

This also adds a refresh_spec.py script to gather some GIT log details to pinpoint exactly which commit to the cel-spec repository was used to extract test cases. The Makefile is also tweaked to create a git.log file that tracks the version details.

@slott56 slott56 requested a review from hudlow May 19, 2026 20:57
This has a new set of `.textproto` files, and revised `.feature` files.

The `wip.toml` is updated to flag new cel-spec test cases as "work-in-process" with the `@wip` tag on the tests.
@slott56 slott56 requested a review from kapilt May 20, 2026 14:19
Copy link
Copy Markdown
Collaborator

@kapilt kapilt left a comment

Choose a reason for hiding this comment

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

Lgtm

@Quantum0uasar
Copy link
Copy Markdown
Collaborator

The refresh_spec.py script addition is a nice touch — having automated GIT log aggregation for tracking cel-spec version alignment will make it much easier to stay in sync with upstream spec changes over time. The @wip tagging approach via tags.toml is also a clean way to separate tests that are aspirational vs. currently passing, rather than just skipping them entirely. Looking forward to seeing these features land!

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