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

Warn and exit when no packages are listed in Aptfile #126

Merged
merged 1 commit into from
Mar 28, 2024

Conversation

colincasey
Copy link
Contributor

@colincasey colincasey commented Mar 20, 2024

This PR fixes an error that occurs if an Aptfile with no packages is used, that was introduced by #115. This case is detected now and a warning is printed advising that the buildpack and Aptfile should be removed before exiting the build.

Fixes #117

@colincasey colincasey requested a review from dzuelke March 20, 2024 17:01
@colincasey colincasey self-assigned this Mar 20, 2024
@colincasey colincasey requested a review from a team as a code owner March 20, 2024 17:01
@colincasey colincasey mentioned this pull request Mar 20, 2024
@edmorley edmorley removed the request for review from a team March 21, 2024 10:37
@colincasey colincasey mentioned this pull request Mar 21, 2024
@colincasey colincasey force-pushed the more_robust_comment_line_check branch from 7b97984 to b24f26b Compare March 22, 2024 15:48
@colincasey colincasey force-pushed the fix_for_aptfile_with_no_packages branch from e8b841d to e5a642c Compare March 22, 2024 15:48
@colincasey colincasey force-pushed the more_robust_comment_line_check branch from b24f26b to c725e16 Compare March 28, 2024 13:50
Base automatically changed from more_robust_comment_line_check to master March 28, 2024 13:51
@colincasey colincasey force-pushed the fix_for_aptfile_with_no_packages branch from e5a642c to 2b1891a Compare March 28, 2024 13:52
@colincasey colincasey merged commit 11e850e into master Mar 28, 2024
4 checks passed
@colincasey colincasey deleted the fix_for_aptfile_with_no_packages branch March 28, 2024 13:53
@colincasey colincasey mentioned this pull request Mar 28, 2024
Frzk pushed a commit to Scalingo/apt-buildpack that referenced this pull request May 13, 2024
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.

Fail when there's no packages to install
2 participants