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

fix windows esi compile #7

Closed
wants to merge 2 commits into from

Conversation

regner
Copy link

@regner regner commented Apr 18, 2018

Added a script that shortens the names of output ESI files. The script runs after all the build steps but before the tests.

Also ran chmod +x on the two build scripts.

Fixes antihax/goesi#1

@regner regner force-pushed the 1-fix-windows-esi-compile branch from 129f3da to 0942c50 Compare April 18, 2018 06:17
@regner
Copy link
Author

regner commented Apr 18, 2018

@antihax think this is good to go. If you want to test it you will need to remove the underscore from the ESI swagger URL as thats broken right now. The tests I ran locally came out good.

@regner
Copy link
Author

regner commented Apr 18, 2018

I may have totally lied about everything... got home and went to do some work on my other project and I am still running into the same issue... but it worked last night after I ran this... so I am confused but will report back later.

@regner regner closed this Sep 5, 2018
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.

1 participant