Skip to content

Test Appveyor integration #5531

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

Closed
wants to merge 40 commits into from
Closed

Conversation

liufengyun
Copy link
Contributor

Based on #5444 . If this works, we could get rid of the project https://github.com/lampepfl/packtest

@liufengyun liufengyun force-pushed the appveyor branch 7 times, most recently from 892016d to 77e2b5d Compare November 28, 2018 13:41
@liufengyun liufengyun force-pushed the appveyor branch 3 times, most recently from ed9528f to b97c07e Compare November 29, 2018 15:33
@liufengyun liufengyun force-pushed the appveyor branch 2 times, most recently from 8e861aa to 5d17f8a Compare December 3, 2018 09:45
@liufengyun liufengyun force-pushed the appveyor branch 3 times, most recently from 9e9dba6 to 0814ca7 Compare December 5, 2018 08:58
In QuoteDriver, previous we have path like the following on windows:

/C:/Users/dotty/My%20Programs/Java/Test/bin/myJar.jar

Credit: https://stackoverflow.com/questions/6164448/convert-url-to-normal-windows-filename-java
"tokens=*" is needed to handle spaces in path
* path-fix:
  Path fix for doc-tool and QuoteDriver
  More path fixes
  Address review
  Refactor without using JFile.separator (thanks @michelou)
  More fix for window path
  Fix window path in ClassPath
  Fix window path
@liufengyun
Copy link
Contributor Author

Close this for now, as Appveyor is too unstable -- unable to find artifacts from time to time.

Wait for Drone windows to be mature and we can have a Drone windows agent on a dedicated machine.

@liufengyun liufengyun closed this Dec 12, 2018
@liufengyun liufengyun deleted the appveyor branch December 12, 2018 08:38
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.

2 participants