Skip to content

feat(license-exceptions): add not-eligible status and fix decision workflow labels - #1495

Open
jeefy wants to merge 3 commits into
cncf:mainfrom
jeefy:main
Open

feat(license-exceptions): add not-eligible status and fix decision workflow labels#1495
jeefy wants to merge 3 commits into
cncf:mainfrom
jeefy:main

Conversation

@jeefy

@jeefy jeefy commented Aug 20, 2026

Copy link
Copy Markdown
Member
  • Change site subtitle to 'Requested license exceptions for CNCF projects' since the database includes denied and not-eligible requests
  • Add 'not-eligible' status for requests that don't qualify for the exception process (e.g., use inconsistent with the CNCF IP Policy), wired through the schema, validator, site filter/badge, and README
  • Fix the decision workflow to trigger on license-exception/approved, license-exception/denied, and license-exception/not-eligible labels (the previous approved/denied labels never existed in the repo, so the workflow had never fired)

…rkflow labels

- Change site subtitle to 'Requested license exceptions for CNCF projects'
  since the database includes denied and not-eligible requests
- Add 'not-eligible' status for requests that don't qualify for the
  exception process (e.g., use inconsistent with the CNCF IP Policy),
  wired through the schema, validator, site filter/badge, and README
- Fix the decision workflow to trigger on license-exception/approved,
  license-exception/denied, and license-exception/not-eligible labels
  (the previous approved/denied labels never existed in the repo, so
  the workflow had never fired)

Signed-off-by: Jeffrey Sica <me@jeefy.dev>
@jeefy
jeefy requested review from joannalee333 and krook August 20, 2026 02:06
jeefy added 2 commits August 20, 2026 09:44
The netlify-preview.yml workflow required NETLIFY_AUTH_TOKEN and
NETLIFY_SITE_ID secrets that were never configured, so it failed on
every run since its introduction. It also could never work for
fork-based PRs, which don't receive repository secrets.

Remove it and re-enable Netlify's native deploy previews by dropping
the 'ignore = "exit 0"' override. Netlify's own credentials handle
fork PRs, and the 'base = license-exceptions/' setting already scopes
preview builds to changes under that directory.

Signed-off-by: Jeffrey Sica <me@jeefy.dev>
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.

1 participant