Enable lint analyze

[JSpiner]

1. enable lint analyze remove following codes

lintOptions {       
       abortOnError false       
}

2. fix lint errors
   • style sdk issue.
   • language support issue.

[JSpiner]

@ParkSangGwon please review it!

[ParkSangGwon]

@JSpiner Please remove last lint error. PermissionBuilder::85: remove if()(@StringRes already check valid resource id)

[JSpiner]

@ParkSangGwon (영어 표현에 한계가 와서 한글로 씁니다 😂 )

무언가 잘못된것같습니다! 확실히 말씀하신 부분은 수정되면 좋다는 생각이 듭니다!!

다만 제 로컬 Inspection Result와 Travis의 lint 테스트에서는 에러를 내지 않습니다!

1. Ted님의 로컬 컴퓨터에 lint 설정 수준이 높게 설정되어있다.
2. Travis CI 및 본인 로컬 컴퓨터에 lint 설정 수준이 낮게 되어있다.
3. Travis에 lint 옵션이 여전히 꺼져있다.

등등의 경우가 있을것같은데, CI를 사용하고있는 시점에서 테스트/정적분석기 결과가 다르게 나오는 원인을 찾아야할것같습니다!

PermissionBuilder::85: remove if()(@stringres already check valid resource id)

이 이슈는 어디서 어떻게 발견하신건가요?

[ParkSangGwon]

@JSpiner Inspect Code(Analyze->Inspect Code)로 돌렸을때 Wrong Resource Type으로 발견됩니다. @JSpiner 님의 lint preferecnes에 Wrong Resource Type는 warning으로 되어있나요?

[JSpiner]

@ParkSangGwon 네! 말씀하신것과 동일한 Inspect Code로 확인했습니다. Wrong Resource Type은 warning으로도 검출되지 않습니다!

lint와 Inspect Code의 결과는 다르다고하는데, 같은 Inspection profile로도 다른 결과가 나오는 이유를 확인한뒤 수정해야할것같습니다 😂

Android Studio 3.0.1버전으로 테스트해봤는데, 다른버전에서도 확인후 공유드리겠습니다.

[ParkSangGwon]

@JSpiner Resolve conflict please.

[MoNTE48]

@ParkSangGwon up! 🙂