Skip to content
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

Add support for .ico files #1740

Merged
merged 1 commit into from
Nov 3, 2016
Merged

Add support for .ico files #1740

merged 1 commit into from
Nov 3, 2016

Conversation

allanchau
Copy link
Contributor

@allanchau allanchau commented Oct 22, 2016

Fixes the build error
ERROR in ./assets/images/favicon.ico Module parse failed: ...sage/assets/images/favicon.ico Unexpected character '' (1:0) You may need an appropriate loader to handle this file type. (Source code omitted for this binary file) @ multi files

@QWp6t QWp6t merged commit e4d8be1 into roots:master Nov 3, 2016
@allanchau allanchau deleted the patch-1 branch November 3, 2016 09:27
ptrckvzn pushed a commit to ptrckvzn/sage that referenced this pull request Dec 1, 2016
* master:
  Remove sidebar defaults
  Replace WP codex links with developer.wordpress.org links
  Remove post formats
  Update yarn.lock [ci skip]
  Update CHANGELOG [ci skip]
  Use new webpack api schema
  Update dependencies
  Update CHANGELOG [skip ci]
  Update alpha references
  Variables organization
  Use $.fn.ready() (reverts 724d550)
  Theme activation updates 'stylesheet' option instead of 'template'
  Reorganize and refactor build routine
  Switch assets manifest plugin
  Add images to assets manifest
  Switch from babel to buble
  Update dependencies & webpack compatibility (BREAKING CHANGES)
  Use http by default (not https) to be consistent with Trellis
  Add support for .ico files (roots#1740)
  Adds Yarn package and lock file. (roots#1744)
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.

2 participants