fix(lints): multiple linter rules #149

[Aadeesh11]

Ref #149.

Describe the changes you have made in this PR -
Enabled linter:

1. unnecessary_parenthesis
2. avoid_redundant_argument_values

Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

[Aadeesh11]

@manjotsidhu my local tests are passing, any idea why CI tests are failing?

[manjotsidhu]

@manjotsidhu my local tests are passing, any idea why CI tests are failing?

Apologies for late reply, thanks for your contribution. CI Tests use flutter 2.8 to build and test so please wait until we resolve this.

[manjotsidhu]

@manjotsidhu my local tests are passing, any idea why CI tests are failing?

You can rebase your changes and CI will pass now

[coveralls]

Pull Request Test Coverage Report for Build 1618864256

• 0 of 2 (0.0%) changed or added relevant lines in 2 files are covered.
• 178 unchanged lines in 6 files lost coverage.
• Overall coverage decreased (-1.2%) to 72.302%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
lib/ui/views/groups/add_assignment_view.dart	0	1	0.0%
lib/ui/views/groups/update_assignment_view.dart	0	1	0.0%

Files with Coverage Reduction	New Missed Lines	%
lib/utils/router.dart	4	57.14%
lib/viewmodels/ib/ib_page_viewmodel.dart	12	53.85%
lib/services/local_storage_service.dart	20	23.26%
lib/viewmodels/ib/ib_landing_viewmodel.dart	32	17.95%
lib/ui/views/ib/ib_landing_view.dart	48	66.21%
lib/ui/views/ib/ib_page_view.dart	62	59.05%

Totals
Change from base Build 1540966382:	-1.2%
Covered Lines:	3879
Relevant Lines:	5365

---

💛 - Coveralls