Skip to content

Fix comments adish#166

Merged
matthew-SG merged 7 commits intomainfrom
Fix-comments-adish
Dec 2, 2025
Merged

Fix comments adish#166
matthew-SG merged 7 commits intomainfrom
Fix-comments-adish

Conversation

@Adish-Singh
Copy link
Copy Markdown
Collaborator

No description provided.

@Peter-Hu12138
Copy link
Copy Markdown
Collaborator

Please resolve merge conflicts before I can merge

} catch (Exception e) {
presenter.prepareFailView("Error adding recipe: " + e.getMessage());
}
catch (Exception ex) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.IllegalCatchCheck> reported by reviewdog 🐶
Catching 'Exception' is not allowed.

} catch (Exception e) {
presenter.prepareFailView("Error deleting recipe: " + e.getMessage());
}
catch (Exception ex) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.IllegalCatchCheck> reported by reviewdog 🐶
Catching 'Exception' is not allowed.

final List<String[]> ingredientsList = new ArrayList<>();
for (Ingredient ingredient : recipe.getIngredients()) {
String[] ingredientData = new String[3];
final String[] ingredientData = new String[3];
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.MagicNumberCheck> reported by reviewdog 🐶
'3' is a magic number.

} catch (Exception e) {
presenter.prepareFailView("Error loading recipes: " + e.getMessage());
}
catch (Exception ex) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.IllegalCatchCheck> reported by reviewdog 🐶
Catching 'Exception' is not allowed.

} catch (Exception e) {
presenter.prepareFailView("Error loading instructions: " + e.getMessage());
}
catch (Exception ex) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.IllegalCatchCheck> reported by reviewdog 🐶
Catching 'Exception' is not allowed.

} catch (Exception e) {
presenter.prepareFailView("Error adding ingredients to grocery list: " + e.getMessage());
}
catch (Exception ex) {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.coding.IllegalCatchCheck> reported by reviewdog 🐶
Catching 'Exception' is not allowed.

Copy link
Copy Markdown
Owner

@matthew-SG matthew-SG left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for adding comments to your code!

@matthew-SG matthew-SG merged commit b0db7c3 into main Dec 2, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants