Skip to content

fix: port ChilliCream license URL fix (#3762) to release/2.0 - #3769

Merged
Souvik Ghosh (souvikghosh04) merged 1 commit into
release/2.0from
fix/chillicream-license-url-release-2.0
Aug 12, 2026
Merged

fix: port ChilliCream license URL fix (#3762) to release/2.0#3769
Souvik Ghosh (souvikghosh04) merged 1 commit into
release/2.0from
fix/chillicream-license-url-release-2.0

Conversation

@souvikghosh04

Copy link
Copy Markdown
Contributor

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 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 returns a 404, causing the pipeline to fail.

The fix (#3762) was merged to main only; release/2.0 still 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: $chiliCreamLicenseMetadataURL now points to website/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.

#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 AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

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 $chiliCreamLicenseMetadataURL to the new website/content/licensing/chillicream-license.md path in ChilliCream/graphql-platform.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@souvikghosh04 Souvik Ghosh (souvikghosh04) moved this from Todo to Review In Progress in Data API builder Aug 12, 2026
@souvikghosh04
Souvik Ghosh (souvikghosh04) merged commit 581da8f into release/2.0 Aug 12, 2026
14 checks passed
@souvikghosh04
Souvik Ghosh (souvikghosh04) deleted the fix/chillicream-license-url-release-2.0 branch August 12, 2026 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Review In Progress

Development

Successfully merging this pull request may close these issues.

4 participants