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

Use actions for CI #38

Merged
merged 29 commits into from
May 2, 2020
Merged

Use actions for CI #38

merged 29 commits into from
May 2, 2020

Conversation

trelau
Copy link
Owner

@trelau trelau commented May 2, 2020

Run the binder during CI

Trevor Laughlin and others added 29 commits January 8, 2020 23:22
* Use binder and add linux support for 7.4.0

* Update azure pipelines script

* Add yes flag for conda install

* Also install occt so binder can run

* Use a separate env
Use GitHub actions instead of azure
@trelau trelau merged commit 06cdb64 into occt_740 May 2, 2020
@trelau trelau deleted the azure_support branch May 2, 2020 05:20
@frmdstryr
Copy link
Contributor

Awesome, looks like its working now!

Could you also pull in the changes to make Font_BRepFont work?

Add a subclass that exposes new and delete
https://github.com/frmdstryr/pyOCCT/blob/github-actions/inc/bind_Font_BRepFont.hxx

Add header (and remove it from excluded classes)
https://github.com/frmdstryr/pyOCCT/blob/github-actions/config.txt#L637

Patch Font_BRepTextBuilder to use the font subclass
https://github.com/frmdstryr/pyOCCT/blob/github-actions/config.txt#L942-L946

I'm not sure if the nodelete option is needed there. I don't get why they decided to make it inherit from a protected class...

@trelau trelau mentioned this pull request May 2, 2020
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