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

Update Dockerfile #394

Merged
merged 2 commits into from
Mar 12, 2023
Merged

Update Dockerfile #394

merged 2 commits into from
Mar 12, 2023

Conversation

Uzurka
Copy link
Contributor

@Uzurka Uzurka commented Mar 12, 2023

Edited Dockerfile so the RUN chmod +x don't show an error anymore

  • This includes major changes (breaking changes)
  • This includes minor changes (minimal usage changes, minor new features)
  • This includes patches (bug fixes)
  • This does not change functionality at all (code refactoring, comments)

Is this related to an issue?

None

Changes made

In the Dockerfile, the RUN chmod +x was made BEFORE the copy of /build to the WORKDIR, so it couldn't run the command because the file doesn't exists.
Changed the position of the RUN command and works again

Confirmations

  • I have updated related documentation (if necessary)
  • My changes use consistent code style
  • My changes have been tested and confirmed to work

Uzurka and others added 2 commits March 12, 2023 10:29
Fix the path so it can stay in the `builder` step, so it doesn't add a layer to the image.
@eartharoid eartharoid merged commit d09598d into discord-tickets:v4 Mar 12, 2023
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.

None yet

2 participants