Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow single buildpack in multiple groups #938

Merged

Conversation

dwillist
Copy link
Contributor

@dwillist dwillist commented Nov 6, 2020

add a check for buildpack repetitions with the same id and version

Signed-off-by: dwillist dthornton@vmware.com

Summary

Fix a bug due to not checking if we have repeated buildpack id & version combinations when validating builtpack configuration.

  • Should this change be documented?
    • Yes, see #___
    • No

Related

Resolves #858

…the same id and version

Signed-off-by: dwillist <dthornton@vmware.com>
@dwillist dwillist requested a review from a team as a code owner November 6, 2020 06:07
Copy link
Member

@dfreilich dfreilich left a comment

Choose a reason for hiding this comment

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

A few nits, but otherwise 👍 from me

looking good

internal/builder/builder.go Outdated Show resolved Hide resolved
internal/builder/builder_test.go Outdated Show resolved Hide resolved
internal/builder/builder_test.go Outdated Show resolved Hide resolved
internal/builder/builder_test.go Outdated Show resolved Hide resolved
@dfreilich dfreilich changed the title fix issue #858, add a check for buildpack repetitions with the same i… Allow single buildpack in multiple groups Nov 6, 2020
@dfreilich dfreilich changed the base branch from main to release/0.15.0-rc2 November 6, 2020 09:49
@jromero jromero added this to the 0.15.0 milestone Nov 9, 2020
@jromero jromero added the type/bug Issue that reports an unexpected behaviour. label Nov 9, 2020
@codecov
Copy link

codecov bot commented Nov 9, 2020

Codecov Report

Merging #938 (4120b77) into release/0.15.0-rc2 (0212268) will increase coverage by 0.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@                  Coverage Diff                   @@
##           release/0.15.0-rc2     #938      +/-   ##
======================================================
+ Coverage               76.55%   76.58%   +0.04%     
======================================================
  Files                     101      101              
  Lines                    4984     4991       +7     
======================================================
+ Hits                     3815     3822       +7     
  Misses                    772      772              
  Partials                  397      397              
Flag Coverage Δ
os_linux 76.58% <100.00%> (+0.04%) ⬆️
os_macos 73.52% <100.00%> (-<0.01%) ⬇️
os_windows 100.00% <ø> (ø)
unit 76.58% <100.00%> (+0.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@jromero jromero merged commit 49ad805 into buildpacks:release/0.15.0-rc2 Nov 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/bug Issue that reports an unexpected behaviour.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

--buildpack flag is throwing incorrect version error when buildpack is listed twice in order
3 participants