fix: port ChilliCream license URL fix (#3762) to release/2.0 - #3769
Merged
Souvik Ghosh (souvikghosh04) merged 1 commit intoAug 12, 2026
Merged
Conversation
#3762) ## Why make this change? - Fixes a 404 in the notice-generation pipeline step "Add Additional Licenses to NOTICE file for Package". - ChilliCream restructured their `graphql-platform` repo and moved the license file from `website/src/basic/licensing/chillicream-license.md` to `website/content/licensing/chillicream-license.md`. The old URL no longer exists and returns a 404, causing the pipeline to fail. ## What is this change? - One-line update in `scripts/notice-generation.ps1`: the `$chiliCreamLicenseMetadataURL` variable is updated to point to the new path `website/content/licensing/chillicream-license.md` on the `main` branch. - The license text itself (`ChilliCream License 1.0`) has not changed — only the path in the repo has moved. ## How was this tested? - [x] Manually verified the new URL returns the raw license text (not HTML) using `Invoke-WebRequest`. ## Sample Request(s) N/A — this is a pipeline script fix with no REST/GraphQL/CLI surface. (cherry picked from commit 8842d08)
Copilot started reviewing on behalf of
Souvik Ghosh (souvikghosh04)
August 12, 2026 07:00
View session
Contributor
There was a problem hiding this comment.
Pull request overview
This PR backports the fix from #3762 to release/2.0 by updating the ChilliCream license raw GitHub URL used by the NOTICE-generation PowerShell script, unblocking the pipeline step that fails due to a 404 after an upstream repo restructure.
Changes:
- Update
$chiliCreamLicenseMetadataURLto the newwebsite/content/licensing/chillicream-license.mdpath inChilliCream/graphql-platform.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
naxing123
approved these changes
Aug 12, 2026
Anusha Kolan (anushakolan)
approved these changes
Aug 12, 2026
Souvik Ghosh (souvikghosh04)
deleted the
fix/chillicream-license-url-release-2.0
branch
August 12, 2026 07:44
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why make this change?
Ports #3762 to
release/2.0.Fixes a 404 in the notice-generation pipeline step "Add Additional Licenses to NOTICE file for Package" when building
release/2.0. ChilliCream restructured theirgraphql-platformrepo and moved the license file fromwebsite/src/basic/licensing/chillicream-license.mdtowebsite/content/licensing/chillicream-license.md. The old URL returns a 404, causing the pipeline to fail.The fix (#3762) was merged to
mainonly;release/2.0still references the old path, so the release/2.0 build fails at the NOTICE step.What is this change?
Cherry-pick of the one-line update in
scripts/notice-generation.ps1:$chiliCreamLicenseMetadataURLnow points towebsite/content/licensing/chillicream-license.md. The license text itself (ChilliCream License 1.0) is unchanged — only the upstream path moved.How was this tested?
Verified the new URL returns the raw license text (not HTML) via
Invoke-WebRequest.