Skip to content

feat: allow buildpacks with empty stacks#1825

Merged
tomkennedy513 merged 1 commit intobuildpacks-community:mainfrom
ctrox:empty-stack
Feb 25, 2025
Merged

feat: allow buildpacks with empty stacks#1825
tomkennedy513 merged 1 commit intobuildpacks-community:mainfrom
ctrox:empty-stack

Conversation

@ctrox
Copy link
Contributor

@ctrox ctrox commented Feb 6, 2025

As of 0.10, stacks are deprecated and optional:

https://github.com/buildpacks/spec/blob/buildpack/v0.10/buildpack.md#buildpacktoml-toml-stacks-array

This allows the stacks to be empty during validation.

Fixes #1730

As of 0.10, stacks are deprecated and optional:

https://github.com/buildpacks/spec/blob/buildpack/v0.10/buildpack.md#buildpacktoml-toml-stacks-array

This allows the stacks to be empty during validation.

Fixes buildpacks-community#1730
@ctrox ctrox requested a review from a team as a code owner February 6, 2025 16:13
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 35.32%. Comparing base (49014e2) to head (831fcdf).
Report is 95 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1825       +/-   ##
===========================================
- Coverage   67.34%   35.32%   -32.03%     
===========================================
  Files         140      257      +117     
  Lines        8886    19965    +11079     
===========================================
+ Hits         5984     7052     +1068     
- Misses       2393    12311     +9918     
- Partials      509      602       +93     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

@natalieparellano natalieparellano left a comment

Choose a reason for hiding this comment

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

I'm not a maintainer, but this looks good to me

@tomkennedy513 tomkennedy513 merged commit 3033eeb into buildpacks-community:main Feb 25, 2025
1 check passed
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.

Update to support 0.10 of the Buildpack API

4 participants