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 CI #226

Merged
merged 14 commits into from
Nov 23, 2023
Merged

Fix CI #226

merged 14 commits into from
Nov 23, 2023

Conversation

trungleduc
Copy link
Member

@trungleduc trungleduc commented Nov 21, 2023

closes #224

Code change

  • Create jupytercad meta package
  • Update version bump scripts
  • Update CI scripts
  • Simplify @jupytercad/opencascade build script
  • Remove Freecad-related test assets.
  • Disable check-python step in the check release workflow.

Copy link
Contributor

Binder 👈 Launch a Binder on branch trungleduc/jupytercad/fix-ci

@trungleduc trungleduc marked this pull request as ready for review November 22, 2023 21:12
@trungleduc
Copy link
Member Author

I'll update the UI test snapshot in the follow-up since the updated Galata bot script needs to be on main

examples/fan.stl Outdated Show resolved Hide resolved
@@ -0,0 +1 @@
export default null as any;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the purpose of this? Is this for TypeScript to copy over the js file into lib/?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file is there to make tsc happy and compile without buiding occ.js into 'src'. It is not copied to lib. The real file in lib is provided by webpack in the occ build process.

Copy link
Member

@martinRenou martinRenou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@martinRenou martinRenou merged commit 21e54e9 into jupytercad:main Nov 23, 2023
7 checks passed
@trungleduc trungleduc deleted the fix-ci branch January 2, 2024 13:46
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.

Fix releaser and setup a bot account?
2 participants