Skip to content

Remove dynamic build of configuration file - #1886

Merged
kpfleming merged 3 commits into
fastly:mainfrom
kpfleming:remove-static-config-builder
Aug 17, 2026
Merged

Remove dynamic build of configuration file#1886
kpfleming merged 3 commits into
fastly:mainfrom
kpfleming:remove-static-config-builder

Conversation

@kpfleming

Copy link
Copy Markdown
Member

Change summary

Commit 468c63b changed the CLI to use the compute-starter-kits.fastly.dev service to obtain starter kit information at runtime (when 'fastly compute init' is executed), so there is no need to have any build- or release-time construction of a static file.

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Commit 468c63b changed the CLI to use
the compute-starter-kits.fastly.dev service to obtain starter kit
information at runtime (when 'fastly compute init' is executed), so
there is no need to have any build- or release-time construction of a
static file.
@kpfleming kpfleming added the Skip-Changelog do not add a changelog entry for this change label Aug 14, 2026
@kpfleming

Copy link
Copy Markdown
Member Author

In preparation for the merge of this PR I've removed "Build static app config" from the 'required checks' list in the branch protection rules.

@kpfleming
kpfleming marked this pull request as ready for review August 14, 2026 19:08
@kpfleming
kpfleming requested a review from a team as a code owner August 14, 2026 19:08
Comment thread .github/workflows/pr_test.yml

@anthony-gomez-fastly anthony-gomez-fastly left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

1 Question, LGTM otherwise

@kpfleming
kpfleming merged commit e9206b2 into fastly:main Aug 17, 2026
9 checks passed
@kpfleming
kpfleming deleted the remove-static-config-builder branch August 17, 2026 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip-Changelog do not add a changelog entry for this change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants