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

Fixes volume error when file in the root folder #5

Merged
merged 10 commits into from
May 4, 2023

Conversation

thomaspoignant
Copy link
Contributor

@thomaspoignant thomaspoignant commented May 4, 2023

Description

When the flag was in the root folder of the repo we had the error

docker: Error response from daemon: create .: volume name is too short, names should be at least two alphanumeric characters.

This PR uses the current folder as a base location to get the flag file.

see failing job: https://github.com/thomaspoignant/test-gha/actions/runs/4880143398/jobs/8707429567

Some other small fixes in this PR:

  • errors were not added to the variable msg.
  • exit 0 is not needed because this is the default behavior in bash.
  • Stick the container to v1.

Changes include

  • Bugfix (non-breaking change that solves an issue)

Checklist

  • I have tested this code

@thomaspoignant thomaspoignant requested a review from a team as a code owner May 4, 2023 07:57
@thomaspoignant thomaspoignant added the bug Something isn't working label May 4, 2023
Copy link
Contributor

@sarvsav sarvsav left a comment

Choose a reason for hiding this comment

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

Minor changes requested.

run-lint.sh Outdated Show resolved Hide resolved
run-lint.sh Outdated Show resolved Hide resolved
Signed-off-by: Thomas Poignant <[email protected]>
@thomaspoignant thomaspoignant merged commit c8726fc into main May 4, 2023
@thomaspoignant thomaspoignant deleted the fix-volume-error branch May 4, 2023 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants