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 168 #175

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Fix 168 #175

wants to merge 13 commits into from

Conversation

jeff-dh
Copy link
Contributor

@jeff-dh jeff-dh commented May 22, 2021

Fixes #168

Based again ontop of #170

The additional commits are
7f65b21
a2e6b05

I had to extact _find_library into helpers.py to be able to import it into objects.py and solidpython.py.

jeff-dh and others added 13 commits May 16, 2021 16:58
	(and added missing newline before EOF)
	- parse_scad_callables now receives a filename -> write test_code to tempfile
	- since the code get written to a file string escapes need to be double escaped ;)
	- corrected syntax in var_with_functions parameters
	- I hope this works for everybody else, but I think it should
	- could this maybe fix the "MacOS mcad issue" from SolidCode#170
	- I'm pretty sure this is what it is supposed to be
	- I would suggest to rename the function to `resolve_scad_filename`
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.

use seems to not handle relative path correctly
1 participant