Skip to content

chore: remove deprecated api-review legacy files#243

Merged
hdamker merged 1 commit intocamaraproject:mainfrom
hdamker:chore/remove-api-review-legacy
Apr 29, 2026
Merged

chore: remove deprecated api-review legacy files#243
hdamker merged 1 commit intocamaraproject:mainfrom
hdamker:chore/remove-api-review-legacy

Conversation

@hdamker
Copy link
Copy Markdown
Contributor

@hdamker hdamker commented Apr 29, 2026

What type of PR is this?

cleanup

What this PR does / why we need it:

Removes the deprecated Fall25-era API Review tooling. The sole upstream consumer (api-review-trigger.yml in camaraproject/ReleaseManagement) was removed earlier; nothing else in the CAMARA repos references the reusable workflow or the v0.6 validator script.

Files removed:

  • .github/workflows/api-review-reusable.yml
  • api-review/docs/CAMARA_API_Review_Workflow_Usage_Guide.md
  • api-review/workflows/api-review-trigger.yml
  • scripts/api_review_validator_v0_6.py

Empty api-review/ and scripts/ directories drop with the files.

Which issue(s) this PR fixes:

Fixes #

Special notes for reviewers:

The README on main no longer references these files (companion PR: docs: add pointer to validation framework documentation).

Changelog input

 release-note
 NONE

Additional documentation

This section can be blank.

@hdamker hdamker requested review from Kevsy and rartych as code owners April 29, 2026 09:55
Copy link
Copy Markdown
Contributor

@rartych rartych left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Contributor

@Kevsy Kevsy left a comment

Choose a reason for hiding this comment

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

LGTM

@hdamker hdamker merged commit d8de430 into camaraproject:main Apr 29, 2026
1 check passed
@hdamker hdamker deleted the chore/remove-api-review-legacy branch April 29, 2026 20:45
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