Skip to content
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

PICS grammar: Adjust for tooling #3

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

cecille
Copy link
Contributor

@cecille cecille commented Jul 5, 2024

Changes the way the PICS conformances are rendered to match with the expected grammar in the tooling.

  • added new macros for PICS used in conformance statements
  • removed otherwise conformances
  • degrade P, D conformances to X
  • fix grammar on not
  • omit choice conformances with + / - / N
  • change optional conformance to use PICS: O style
  • changed macros to use the squished together styles

Still left to do:

  • fix or and and operations so they don't mess up the tables
  • add :M to mandatory items with pics dependencies
  • command PICS
  • split PICS into separate file so it can be generated
  • check grammar with Agustin

For later:
I like screaming snake case better and not just beccause the name is awesome. Let's use that, but we'd need to search/replace the whole test plan.

cecille and others added 2 commits July 5, 2024 17:00
Changes the way the PICS conformances are rendered to match
with the expected grammar in the tooling.

- added new macros for PICS used in conformance statements
- removed otherwise conformances
- degrade P, D conformances to X
- fix grammar on not
- omit choice conformances with + / - / N
- change optional conformance to use PICS: O style
- changed macros to use the squished together styles

Still to do:
- command PICS
- split PICS into separate file so it can be generated
- check grammar with Agustin

For later:
I like screaming snake case better and not just beccause the name
is awesome. Let's use that, but we'd need to search/replace the
whole test plan.
hasty added a commit that referenced this pull request Oct 29, 2024
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.

2 participants