Skip to content

feat(GAT-8807): Adding licenses to generated SDKs. Defaulting to MIT. - #1882

Open
andy-yates-hdr wants to merge 3 commits into
devfrom
feat/GAT-licenses-for-sdk
Open

andy-yates-hdr wants to merge 3 commits into
devfrom
feat/GAT-licenses-for-sdk

Conversation

@andy-yates-hdr

@andy-yates-hdr andy-yates-hdr commented Oct 19, 2026 •

Copy link
Copy Markdown

Claude Sonnet 5.5 used to explore solution for adding license. Originally tested on live but actually no need. Run locally against gateway-api.

Screenshots (if relevant)

None

Describe your changes

Added the MIT license as a template and will be copied into the resultant SDKs. Result is we distribute licensed SDKs. Subsequent issue needs to be raised with registry's SDKs but will only do that if this feature is approved.

Issue ticket link

https://hdruk.atlassian.net/browse/GAT-8807

Environment / Configuration changes (if applicable)

None

Requires migrations being run?

None

If not using the pre-push hook. Confirm tests pass:

No changes to codebases to force test problems. Though pest, phpstan, phpcs and lint run. Behat not run.

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have added appropriate unit tests
  • I have created mocks for unit tests (where appropriate)
  • I have added appropriate Behat tests to confirm AC (if applicable)
  • I have added Swagger annotations for new endpoints (if applicable)
  • I have added audit logs for new operation logic (if applicable)
  • I have added new environment variables to the .env.example file (if applicable)
  • I have added new environment variables to terraform repository (if applicable)

Claude Sonnet 5.5 developed but with manual review and brief refactoring. Tested on live and license appears in generated codebases.
@gh-actions-pipelines-app

Copy link
Copy Markdown

🚨 Invalid PR title format!

Your PR title must follow one of these formats:

  • feat(GAT-1234): Your title
  • fix!(GAT-5678): Breaking change
  • RELEASE: vX.Y.Z

Please update your PR title accordingly. 😊

@andy-yates-hdr andy-yates-hdr changed the title feat(GAT-8807) Adding licenses to generated SDKs. Defaulting to MIT. feat(GAT-8807): Adding licenses to generated SDKs. Defaulting to MIT. Oct 19, 2026
@gh-actions-pipelines-app

Copy link
Copy Markdown

🎉 Great job! Your PR title follows the correct format. 🚀

Comment thread scripts/generate-sdks.sh Outdated
- gateway-api is being relicensed to MIT as per our expectations
- Added MIT to the Controller OA license
- generate-sdks now does more to make sure licenses are clear in all config/package metadata as needed

Claude Sonnet 5.5 used to generate the sdks work. Code reviewed.

This branch was successfully deployed

1 active deployment
dev — a805d1d7 Deployed Oct 23, 2026 by andy-yates-hdr via test #6987
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