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

hotfix(server) fix dockerfile build failure #2555

Merged
merged 2 commits into from
Apr 12, 2022

Conversation

abrl91
Copy link
Member

@abrl91 abrl91 commented Apr 7, 2022

Issue Number: #2554

PR Details

remove unnecessary test file that its import
add to docker ignore mock files
add -- --scope @amplication/server --include-dependencies to npm run prisma:generate

  • Tests for the changes have been added
  • npm test doesn't throw any error

IMPORTANT: Please review the CONTRIBUTING.md file for detailed contributing guidelines.

@abrl91 abrl91 requested a review from mshidlov April 7, 2022 13:35
@abrl91 abrl91 self-assigned this Apr 7, 2022
@mshidlov mshidlov merged commit 0de54b3 into master Apr 12, 2022
@mshidlov mshidlov deleted the hotfix/2554-amplication-server-dockerfile branch April 12, 2022 11:50
@tupe12334 tupe12334 linked an issue Apr 14, 2022 that may be closed by this pull request
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.

🐛 Bug Report: amplicatoin-server dockerfile build failed
2 participants