-
Notifications
You must be signed in to change notification settings - Fork 28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: code cleanup #225
chore: code cleanup #225
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Ok, so this has a bunch of conflicts, leaving to @miquelbeltran to have a look at resolving before merge. |
@@ -23,8 +24,7 @@ app.set("view engine", "ejs"); | |||
// Add the Raygun breadcrumb Express handler | |||
app.use(raygunClient.expressHandlerBreadcrumbs); | |||
|
|||
// uncomment after placing your favicon in /public | |||
// app.use(favicon(__dirname + '/public/favicon.ico')); | |||
app.use(favicon(__dirname + "/public/favicon.ico")); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will it throw an error if no favicon is present?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A favicon has been included in the repo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one thing to consider. Thank you for implementing my feedback 😊
Co-authored-by: Sumitra Manga <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the quick turn around!
chore: code cleanup
Description 📝
Type of change
Updates
Test plan 🧪
Author to check 👓
Reviewer to check ✔️