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

Docs site fixes #687

Closed
wants to merge 1 commit into from
Closed

Docs site fixes #687

wants to merge 1 commit into from

Conversation

shawnbot
Copy link
Contributor

@shawnbot shawnbot commented Feb 21, 2019

  1. We don't need now as a dependency since we're using primer/deploy. Turns out we do: Run now from this repo's working dir, not via npx deploy#10
  2. The files list in now.json is missing "static", which prevents the favicon and some other images from being uploaded to the deployment.

@shawnbot
Copy link
Contributor Author

You can see that this fixes the favicon issue on the branch alias. ✨

@shawnbot shawnbot marked this pull request as ready for review February 21, 2019 17:14
@shawnbot shawnbot mentioned this pull request Feb 21, 2019
12 tasks
@shawnbot
Copy link
Contributor Author

Merged locally into #688. 🚀

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.

None yet

1 participant