-
Notifications
You must be signed in to change notification settings - Fork 0
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
chore(packages): add ESLint, vscode extensions, and github workflows #145
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
extra points for making sure lint runs during the |
Can't tell if the vercel deployment is failing because this is a draft. It's ready for review aside from the deploy issue. |
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.
Love it! Let's fix the vercel deploys and then I'm all in
@@ -0,0 +1,27 @@ | |||
name: Lint Web Embeds |
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.
Amazing, thank you!
@@ -1,19 +1,21 @@ | |||
<!DOCTYPE html> | |||
<!doctype html> |
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.
my heart breaths easy now
@vinnyjth The vercel deploy is passing now. |
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.
🎉
🐛 Issue
Switching between repos was causing havoc due to differences in lint/prettier setups
✏️ Solution
web-embeds
in line with lint/prettier setup on Apollos Adminvscode
extensions config as well to make context switching easier🔬 To Test
yarn
in theweb-embeds
folderyarn format
and see no changesyarn lint
and see no changes📸 Screenshots