WIP: Add additional CI checks (!4) · Merge requests · Server Tools / gitlab-runner-do-monitor · GitLab

Snippets Groups Projects

Closed Erick Hitter requested to merge add/more-ci into master 6 years ago

Handle false report due to context.TODO()

Edited 6 years ago by Erick Hitter

Activity

Erick Hitter added 1 commit 6 years ago
added 1 commit

212c7161 - Fix templating

Compare with previous version
Erick Hitter added 1 commit 6 years ago
added 1 commit

2100178b - Fix templating

Compare with previous version
Erick Hitter added 1 commit 6 years ago
added 1 commit

9983cb55 - Stop being dumb

Compare with previous version
Erick Hitter added 1 commit 6 years ago
added 1 commit

d219df79 - Go back to 7d6083d8 and drop any dependency installation in code quality test

Compare with previous version
Erick Hitter added 1 commit 6 years ago
added 1 commit

b20ae0f8 - Empty isn't enough

Compare with previous version
Erick Hitter unmarked as a Work In Progress 6 years ago

unmarked as a Work In Progress
Erick Hitter @ethitter 6 years ago

Author Owner

context.TODO() is erroneously being flagged as a todo.
Erick Hitter marked as a Work In Progress 6 years ago

marked as a Work In Progress
Erick Hitter changed the description 6 years ago

changed the description
Erick Hitter added 1 commit 6 years ago
added 1 commit

89cab96b - Add JUnit report

Compare with previous version
Erick Hitter added 1 commit 6 years ago
added 1 commit

fd59ed07 - Fix report output path

Compare with previous version
Erick Hitter assigned to @ethitter 5 years ago

assigned to @ethitter
Erick Hitter @ethitter 5 years ago

Author Owner

Closing as checks aren't reported unless I upgrade to an Ultimate license. Additionally, per https://docs.gitlab.com/ee/user/application_security/sast/, the implementation changed in 11.9.
Erick Hitter closed 5 years ago

closed

Please register or sign in to reply