-
Notifications
You must be signed in to change notification settings - Fork 25
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
slack deploy fails on Windows but is successful on Linux #316
Comments
We've had a couple of reports (mostly internally) about the bundling failing more often on Windows 😞 Are you using any third party dependencies in the Another question: does |
Hey @filmaj, Yes, In terms of third-party integrations we are relying on the
|
Hey @djbensono, thanks for the details! Following your steps, I was able to recreate the error on a Windows machine:
I receive the same error when I run the build hook directly, instead of using the deploy command. So that may help us narrow down the culprit. Bundling worked okay on macOS. |
Any difference if you use the |
Using
I should also note that using |
OK, thanks for sharing. As I mentioned, we've had a few reports of this. I intend to experiment with adding another fallback to |
The
deno-slack
versions"deno-slack-sdk/": "https://deno.land/x/deno_slack_sdk@2.10.0/",
"deno-slack-api/": "https://deno.land/x/deno_slack_api@2.4.0/",
Deno runtime version
deno 1.43.2 (release, x86_64-pc-windows-msvc)
v8 12.4.254.12
typescript 5.4.5
OS info
Microsoft Windows [Version 10.0.22631.3447]
Describe the bug
slack deploy
gives the following error when ran on a Windows based machine:However, when executed as part of a Github Action running on a Ubuntu 22.04.4 LTS agent -
slack deploy
is executed successfully and the application is deployed to my Slack workspace.I have seen similar issues regarding problems with Windows and these point to having spaces within the path or linting issues from within Deno. There are no spaces in my full folder path and
deno lint
returns no linting errors.The text was updated successfully, but these errors were encountered: