-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
🐛 Bug Report: Use Default Build Commands #6428
Comments
Hello maintainers, |
@ItzNotABug, thanks for raising this! Let me bring it up with the team. |
Why would you not pass in a build command 🤔 This doesn't feel like a bug. It's expected that if you don't build, you can't run a function 😅 |
@gewenyu99 The build commands are marked optional, if I remember correctly. My suggestion was to check if there's a command passed. If yes then process that, if not then it wouldn't hurt to use a default command. Its pretty comman where someone would just forget adding the commands when creating a function. This would save a trip to add the commands again and redeployment. |
👟 Reproduction steps
If there are no build commands passed, e.g.
npm install
, the build will succeed but the execution will fail because the dependencies are not loaded.See -
This makes the
optional
command, not so optional as we must pass a build command.👍 Expected behavior
Runtime should check & handle this behaviour and simply call a relevant build command if one doesn't exist for the function.
👎 Actual Behavior
Execution fails.
🎲 Appwrite version
Version 1.4.x
💻 Operating system
Linux
🧱 Your Environment
No response
👀 Have you spent some time to check if this issue has been raised before?
🏢 Have you read the Code of Conduct?
The text was updated successfully, but these errors were encountered: