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

Feat/javascript to python #2

Closed
wants to merge 5 commits into from
Closed

Conversation

eideegeo
Copy link

@eideegeo eideegeo commented Mar 3, 2025

No description provided.

@eideegeo eideegeo self-assigned this Mar 3, 2025
@eideegeo eideegeo marked this pull request as draft March 3, 2025 11:31
@eideegeo eideegeo force-pushed the feat/javascript-to-python branch from 751c191 to aab620b Compare March 3, 2025 13:12
@eideegeo eideegeo force-pushed the feat/javascript-to-python branch from aab620b to ff59ca6 Compare March 3, 2025 14:03
@eideegeo eideegeo marked this pull request as ready for review March 3, 2025 14:06
@eideegeo eideegeo requested a review from alexismanin March 3, 2025 14:06
eideegeo and others added 3 commits March 3, 2025 15:24
The client factory was embedded in server main source. However, it was not used by it, and it import some heavyweight dependencies (graalvm js engine, jython).
As it serves for test purposes, we put it in project test fixtures, so we can use it in tests, and we will be able to use it in future dependant test modules.
Disable it on pull requests, as it fails when doing a pull request from a fork.
@alexismanin
Copy link

Obsolete, we directly merged into pdssp repository

@alexismanin alexismanin closed this Mar 4, 2025
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