FalckJoshua / DIT113-System-Development-Locus-Imperium

Other
0 stars 0 forks source link

Fixed the linting error #46 - [merged] #94

Closed FalckJoshua closed 7 months ago

FalckJoshua commented 1 year ago

In GitLab by @carlosca on May 2, 2023, 18:37

Merges 46-fix-android-app-code-linting-errors -> main

It was a simple fix, just had to delete one line.

Closes #46

FalckJoshua commented 1 year ago

In GitLab by @carlosca on May 2, 2023, 18:37

requested review from @cjoshua

FalckJoshua commented 1 year ago

In GitLab by @cjoshua on May 2, 2023, 18:40

enabled an automatic merge when the pipeline for 0a6b7a823d58ea2752f7e6b62b7e3851f9300b1b succeeds

FalckJoshua commented 1 year ago

In GitLab by @cjoshua on May 2, 2023, 18:40

Tested the MQTT functionality without the removed line, functionality stayed the same. Good job spotting dead code!

FalckJoshua commented 1 year ago

In GitLab by @cjoshua on May 2, 2023, 18:40

approved this merge request

FalckJoshua commented 1 year ago

In GitLab by @vasilena on May 2, 2023, 18:44

approved this merge request

FalckJoshua commented 1 year ago

In GitLab by @cjoshua on May 2, 2023, 18:53

mentioned in commit 8191ae1be1f244f26db1059b9ebeeb8cacdbd352