search

cdli-gh / cdli-mobile-app

MIT License
4 stars 8 forks source link

Resolves all warnings #10

Closed MrUnfunny closed 3 years ago

MrUnfunny commented 3 years ago

Resolves #9. Splits the task in 3 parts with 1 commit each. Changes are lot more than intended due to formatting by IDE. I used line length of 80 since different files had different line lengths for formatting. Should we define a default line length for the whole project as well?

MrUnfunny commented 3 years ago

@NishealJ please take a look. If changes due to formatting are making it difficult to review, I can send a PR without any formatting

NishealJ commented 3 years ago

Cool, lgtm. All the needed functionalities work. Thanks @MrUnfunny