lbry-android icon indicating copy to clipboard operation
lbry-android copied to clipboard

Linting errors hinder ability to build repository

Open clay53 opened this issue 3 years ago • 4 comments

The Issue

I keep getting linting errors when trying to build this app. I'm fairly sure it's not just me. This forces me to have to put lintOptions { abortOnError false } in the android options of app/build.gradle to be able to build and install the app. Can these issues be fixed or should this workaround be added to source?

Steps to reproduce

  1. Clone repo
  2. Open it in Android Studio
  3. Set up stuff
  4. Try to run

Expected behaviour

I should be able to build the project without having to do a workaround.

Actual behaviour

I have to do a workaround so I can build the project.

System Configuration

  • Device: pop!os latest

Screenshots

html file of errors included as .txt file (to make github happy) lint-results.txt

Acceptance Criteria

  1. Workaround added to source OR linting errors fixed

clay53 avatar Dec 13 '20 01:12 clay53

I am currently working on fixing linter warnings for some files, beginning with MainActivity, as they make very hard to see other markers on the scroll bar. I am doing it on a git branch and from time to time, as it es not going to fix any bug.

These warnings aren't making impossible to build, though.

The only problem I have when building is that I have to disable every line related to GMS on app/build.gradle and add a twitter.properties file which shouldn't be added to any commit.

kekkyojin avatar Dec 13 '20 12:12 kekkyojin

I am currently working on fixing linter warnings for some files, beginning with MainActivity, as they make very hard to see other markers on the scroll bar. I am doing it on a git branch and from time to time, as it es not going to fix any bug.

These warnings aren't making impossible to build, though.

The only problem I have when building is that I have to disable every line related to GMS on app/build.gradle and add a twitter.properties file which shouldn't be added to any commit.

Removing lines related to GMS did not change me having to disable linting error aborts. I've never have issues related to GMS before so I'm not sure what's happening here.

clay53 avatar Dec 14 '20 17:12 clay53

I called out the missing twitter.properties file in the instructions in #997 but it seems that it hasn't been fixed yet.

clay53 avatar Dec 14 '20 18:12 clay53

I'm no longer having issues with this.

clay53 avatar Dec 27 '20 08:12 clay53