-
Notifications
You must be signed in to change notification settings - Fork 145
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
Validate DAG schema on Monaco #659
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #659 +/- ##
==========================================
- Coverage 66.16% 66.02% -0.15%
==========================================
Files 53 53
Lines 4156 4174 +18
==========================================
+ Hits 2750 2756 +6
- Misses 1186 1197 +11
- Partials 220 221 +1
Continue to review full report in Codecov by Sentry.
|
Hi @vhespanha, thank you so much for your fantastic work and excellent suggestions! Really appreciate it, I see that this is a hard issue to handle. Regarding Schema and Definition Updates: Regarding Monaco Editor Integration: Regarding Error Handling: I did try to test the changes on my end, but it seems the schema validation isn't triggering in my environment. This could potentially be an issue on my side. I'm curious - how is it functioning in your development environment? Could you share some details or screenshots about how you're seeing it work? Once again, thank you for your hard work! |
This PR implements DAG schema validation in the Monaco Editor, like suggested here: issue #652
Changes
Schema and Definition Updates
definition
struct and JSON schema to accept both array and string representations of theParams
field.Monaco Editor Integration
Error Handling
Next steps
The current implementation is functional but i'm considering it a draft since the approach of modifying the JSON schema to handle dual
Params
representation probably needs review, I'm willing to work on better diagnostics if all is right with the current approach.