Skip to content

Conversation

@greg-a-smith
Copy link
Contributor

Description

The gh-pages package uses simple authentication by default which was causing the fundamental-bot user to fail authentication when attempting to publish the docs. I added a repo argument that embeds the GitHub token so we can push to the gh-pages branch.

fixes #352

@greg-a-smith greg-a-smith requested a review from a team February 13, 2019 18:17
Copy link
Contributor

@mobot11 mobot11 left a comment

Choose a reason for hiding this comment

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

🚢

@greg-a-smith
Copy link
Contributor Author

Per feedback, I made some changes to keep the repo URL construction out of package.json.

Copy link
Contributor

@bcullman bcullman left a comment

Choose a reason for hiding this comment

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

looks good. 🚢

@greg-a-smith greg-a-smith merged commit de9fc8f into master Feb 14, 2019
@greg-a-smith greg-a-smith deleted the chore/gh-pages-publish branch February 14, 2019 03:19
greg-a-smith added a commit that referenced this pull request Mar 5, 2019
* Added repo arg to the gh-pages call

* Changed deploy script definition to just run gh-pages so the repo argument could be passed in
ademidun added a commit to atilatech/client-web-app that referenced this pull request Aug 31, 2019
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.

The deploy step to gh-pages is failing on Travis

5 participants