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

Js translator #620

Closed
wants to merge 9 commits into from
Closed

Js translator #620

wants to merge 9 commits into from

Conversation

davinov
Copy link
Member

@davinov davinov commented Aug 28, 2020

As a side-project, I started to develop a translator that transform pipelines into js functions.
This enables the playground to run client-side only. Once complete (all steps supported), it can also be used as reference for how transformations are expected to work (with tests) or to preview transformations when the database/server is not available.

Closes #519

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

This pull request has been automatically deployed to FeaturePeek. 😄
Your deployments will be kept up-to-date with this pull request's latest changes.

app link
playground https://peek.run/1jwhv9dj/playground
storybook https://peek.run/1jwhv9dj/storybook

Please read our docs for more configuration details.

@davinov davinov force-pushed the js-translator branch 3 times, most recently from 8f17bda to 6c9976e Compare August 28, 2020 09:49
Now that the playground can work client-side only, it's interesting to
deploy it as a static website for easy reviews.
@codecov
Copy link

codecov bot commented Aug 28, 2020

Codecov Report

Merging #620 into master will increase coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #620      +/-   ##
==========================================
+ Coverage   97.68%   97.72%   +0.04%     
==========================================
  Files         143      144       +1     
  Lines        3713     3783      +70     
  Branches      534      550      +16     
==========================================
+ Hits         3627     3697      +70     
  Misses         86       86              
Impacted Files Coverage Δ
lib/translators/index.ts 100.00% <0.00%> (ø)
lib/translators/js.ts 100.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3d3b251...a46d75f. Read the comment docs.

@adimascio adimascio temporarily deployed to weaverbird-js-translato-io4fgm August 28, 2020 10:18 Inactive
@davinov
Copy link
Member Author

davinov commented Mar 18, 2022

Closing this PR, as we've taken a different route (to implement backends in python)

@davinov davinov closed this Mar 18, 2022
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.

JavaScript backend
2 participants