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

eslint + updated readme #41

Merged
merged 8 commits into from
Nov 13, 2022
Merged

eslint + updated readme #41

merged 8 commits into from
Nov 13, 2022

Conversation

abheektripathy
Copy link
Contributor

solves #27
added documentation to run frontend locally.

@vercel
Copy link

vercel bot commented Nov 11, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
eye ❌ Failed (Inspect) Nov 13, 2022 at 3:35PM (UTC)

@vercel vercel bot temporarily deployed to Preview November 11, 2022 10:45 Inactive
@0x0elliot
Copy link
Contributor

0x0elliot commented Nov 11, 2022

please do pre-commit run --all-files and then add changes and push.

@vercel vercel bot temporarily deployed to Preview November 11, 2022 14:16 Inactive
@vercel vercel bot temporarily deployed to Preview November 11, 2022 15:33 Inactive
@0x0elliot
Copy link
Contributor

you will have to install the commands you're using in the actions containers. the command utility you're using here hasn't been installed and thus the error is coming.

@0x0elliot
Copy link
Contributor

also, can you please change this to be "Eye" instead of "urNode".

@0x0elliot
Copy link
Contributor

let me see how i can help fix this

@vercel vercel bot temporarily deployed to Preview November 13, 2022 15:32 Inactive
@0x0elliot
Copy link
Contributor

i think i might have figured out the issue

@vercel vercel bot temporarily deployed to Preview November 13, 2022 15:35 Inactive
@0x0elliot
Copy link
Contributor

0x0elliot commented Nov 13, 2022

very cool. it works now. will fix eslint errors in a fix spree now.

@0x0elliot 0x0elliot merged commit 3aedbe6 into ArgusOSS:master Nov 13, 2022
@0x0elliot 0x0elliot mentioned this pull request Nov 13, 2022
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