Skip to content
This repository has been archived by the owner on May 14, 2020. It is now read-only.

Remove redundant try-catch #171

Merged
merged 1 commit into from
Jan 10, 2020

Conversation

thewilkybarkid
Copy link
Contributor

NotAnArticle will never be thrown due to

const id = clownface({ dataset: request.dataset }).has(rdf.type, schema.Article).term;
if (!id) {
throw new createHttpError.BadRequest(`No ${termToString(schema.Article)} found`);
}

Found by #167.

@thewilkybarkid thewilkybarkid added the 📦 Refactor A code change that neither fixes a bug nor adds a feature label Jan 10, 2020
@thewilkybarkid thewilkybarkid requested a review from a team as a code owner January 10, 2020 15:50
@thewilkybarkid thewilkybarkid merged commit 41a1ae7 into libero:master Jan 10, 2020
@thewilkybarkid thewilkybarkid deleted the no-article-found branch January 10, 2020 16:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
📦 Refactor A code change that neither fixes a bug nor adds a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants