Make the default favicon actually a default

mattdesl / budo

:clapper: a dev server for rapid prototyping

MIT License

2.17k stars 106 forks source link

Closed benburrill closed 4 years ago

benburrill commented 4 years ago

Fixes #249 in the first way I suggested

benburrill commented 4 years ago

The CI tests seem to be broken. To test this commit, I did the following:

Ran npm run test, which passed (after killing a server to appease those annoying tests that check exact port numbers)
Ran npm run start, which (probably duplicating a test), showed no regression in loading the null favicon
Tried it with my game, which loaded my favicon as expected.

mattdesl commented 4 years ago

Thanks!