Apply suggestions from code review
Co-authored-by: Fredrik Adelöw <freben@gmail.com> Co-authored-by: Adam Harvey <adam.harvey@dxc.com> Signed-off-by: Patrik Oldsberg <poldsberg@gmail.com>
This commit is contained in:
@@ -129,7 +129,9 @@ export class TodoScmReader implements TodoReader {
|
||||
})),
|
||||
);
|
||||
} catch (error) {
|
||||
this.logger.error(`Failed to parse TODO in ${url}, ${error}`);
|
||||
this.logger.error(
|
||||
`Failed to parse TODO in ${url} at ${file.path}, ${error}`,
|
||||
);
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
@@ -51,7 +51,7 @@ export class TodoReaderService implements TodoService {
|
||||
|
||||
async listTodos(req: ListTodosRequest): Promise<ListTodosResponse> {
|
||||
if (!req.entity) {
|
||||
throw new InputError('entity filter is required to list todos');
|
||||
throw new InputError('Entity filter is required to list TODOs');
|
||||
}
|
||||
const entity = await this.catalogClient.getEntityByName(req.entity);
|
||||
if (!entity) {
|
||||
|
||||
Reference in New Issue
Block a user