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

execute test with compile #493

Closed
wants to merge 2 commits into from

Conversation

mmurooka
Copy link
Member

@mmurooka
Copy link
Member Author

WIP
some tests, which loads other files like https://github.com/euslisp/jskeus/blob/master/irteus/test/irteus-demo.l#L5 becomes error when compiling and testing.

k-okada added a commit to k-okada/jskeus that referenced this pull request Aug 10, 2018
override euslisp#493, use eval-when not to run test code on compile time
k-okada added a commit to k-okada/EusLisp that referenced this pull request Aug 12, 2018
- override euslisp/jskeus#493, use eval-when not to run test code on compile time
k-okada added a commit to k-okada/EusLisp that referenced this pull request Aug 12, 2018
- override euslisp/jskeus#493, use eval-when not to run test code on compile time
k-okada added a commit to k-okada/EusLisp that referenced this pull request Aug 12, 2018
- override euslisp/jskeus#493, use eval-when not to run test code on compile time
@k-okada
Copy link
Member

k-okada commented Sep 11, 2018

You can use eval-when to not execute when compile the source, 6cf3625 and https://github.com/euslisp/jskeus/pull/523/files if you're ok wich tests commits, please close this PR

@k-okada
Copy link
Member

k-okada commented Sep 11, 2018

Btw, did you have ploblem like
euslisp/EusLisp#324 when you try to test with compiled code?

@mmurooka
Copy link
Member Author

Thank you. I close this PR.

Btw, did you have ploblem like
euslisp/EusLisp#324 when you try to test with compiled code?

I did not notice that problem.

@mmurooka mmurooka closed this Sep 12, 2018
@mmurooka mmurooka deleted the add-compile-test-all branch September 12, 2018 03:58
@k-okada k-okada mentioned this pull request Nov 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants