-
Notifications
You must be signed in to change notification settings - Fork 52
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
fixes linting errors #92
base: main
Are you sure you want to change the base?
Conversation
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.
Hi @ovpv you have merge conflicts in your PR. check them out? rest LGTM
Hey @pythongiant , I have resolved the conflicts. Please check and let me know if there are any further issues. |
hey @pythongiant , After the latest pull from the main branch, The build checks are failing. I am not sure why that's happening? can you help? |
Looks like checks are passing now, Good work |
hey @pythongiant , Does this usually take this much time to merge a PR in this repository or is it on hold due to some issue? if so please let me know. |
Fixes #50
Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!
@publiclab/reviewers
or the user who published the issue for help, in a comment belowPlease be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software
Thanks!