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

Travis [CI] Is Not Happy - Code Style Errors #313

Open
misspellted opened this issue Feb 7, 2020 · 4 comments
Open

Travis [CI] Is Not Happy - Code Style Errors #313

misspellted opened this issue Feb 7, 2020 · 4 comments

Comments

@misspellted
Copy link
Contributor

As a recent new contributor to LBRY (.tech in particular), I was saddened to see, as part of the process for pull requests (#312 for example), that Travis CI failed the build.

I am currently in the software engineering field, more specifically devops-y releated, and seing a failed build makes me sad. So, while it is a little difficult to describe the Expected and Actual Behaviors, I did notice a few entries that appear to be correctable code style errors (without digging into the source right away before creating this issue):

  • no-multi-spaces
  • prefer-const
  • no-useless-return
  • arrow-spacing

As someone not quite versed in JavaScript, I am not sure I would be sufficient in resolving the few prefer-promise-reject-errors entries, so I am thinking of just tackling the "simple"-looking errors for now, in hopes to at least reduce the errors reported. I don't know if the 25 style errors are the maximum the linter/checker will report - maybe there are additional ones after fixing the ones already reported?

I don't think it would hurt [too much] to at least get started in tackling the bugs, to get Travis to a happy/happier state.

@cassidypignatello
Copy link
Contributor

cassidypignatello commented Apr 11, 2020

Opened a PR for this at #317. All eslint errors have been resolved and Travis looks happy. Please let me know if there's anything else I should do in order to have this accepted.

@misspellted
Copy link
Contributor Author

Travis looks happy! Thanks @cassidypignatello!

Since lbry-bot assigned the PR to @kauffj, it'll just be a quick review of the changes and most likely an approval will be granted to accept the PR.

@kauffj
Copy link
Member

kauffj commented Apr 14, 2020

Thanks to both of you! Kind of an ugly code base so your work is extra appreciated @cassidypignatello 😛

If you want to work on some nicer (and deeper) JavaScript, come join us on lbry.tv / lbry-desktop!

https://github.com/lbryio/lbry-desktop

@cassidypignatello
Copy link
Contributor

Happy to help! I'm a stickler for code style so this sort of thing is right up my alley. 😉

I'll check out some of other issues on lbry-desktop as well, I'd definitely like to contribute more!

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

No branches or pull requests

3 participants