-
Notifications
You must be signed in to change notification settings - Fork 32
407 Add test workflow for leapfrogai_ui #439
407 Add test workflow for leapfrogai_ui #439
Conversation
✅ Deploy Preview for leapfrogai-docs canceled.
|
…rkflow-for-svelte-code-e2e
I disabled the 'no-undef' rule for eslint because Typescript is already checking for this and gives no errors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a question and some minor things!
all lgtm pending @justinthelaw's feedback being resolved. |
…rkflow-for-svelte-code-e2e
fix conflicts then lgtm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two small change requests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
After the workflows were added, linting changes were applied to this PR to get the UI compliant with the linter/formatter