-
Notifications
You must be signed in to change notification settings - Fork 889
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
Postinstall fail on windows #275
Comments
We should have a proper way to test this. VM on linux/macOS? XD @ZakGriffith are you still on windows? If so, could you try? Also, maybe we should just un-gitignore
Thoughts? |
Yeah maybe that's better. We'll need to worry a lot less about build pipelines and the postinstall script etc. When we make changes to the contract for testing stuff, we need to be mindful of not committing that one anayway, might as well keep an extra eye out for changes to |
Ok, let's go with it and see how it feels. |
i'm on windows and just needed to change this. not sure if it'll affect mac users though |
This issue will be fixed with #278 In any case, we need to have keep this in mind for future scripts :) |
Closing this in favour of #278 |
Its seems that there is some problem when resolving the path while creating the generated file.
Sharing below a SS from the TG group facing an issue :
Its seems its resolving c:\c:\ instead of c:\
The text was updated successfully, but these errors were encountered: