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

ci: update .travis.yml #37

Merged
merged 1 commit into from
Jun 8, 2018
Merged

ci: update .travis.yml #37

merged 1 commit into from
Jun 8, 2018

Conversation

abhaychawla
Copy link
Member

Description

Update travis configuration for auto deployment of pull requests to gh-pages.

Related issues and discussion

#4

Screenshots, if any

Checklist

Please make sure these boxes are checked before submitting your pull request - thanks!

  • If you have multiple commits please combine them into one commit by squashing them.

  • Read and understood the contribution guidelines at web-app/.github/CONTRIBUTING.md.

Update travis configuration for auto deployment of pull requests to
gh-pages.

Fixes openMF#4
@abhaychawla
Copy link
Member Author

GITHUB_TOKEN environment variable needs to be set in repository settings for travis before merge.

- ng build --prod

after_success:
- ng build --prod --base-href="/web-app/"
Copy link
Contributor

Choose a reason for hiding this comment

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

You might want to set --base-hre="https://openmf.github.io/web-app/"
It won't work otherwise once you push to gh-pages

Copy link
Member Author

Choose a reason for hiding this comment

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

No I think its not necessary because I tried this on one of my repositories and it worked fine.

@abhaychawla abhaychawla merged commit 7db9148 into openMF:master Jun 8, 2018
@abhaychawla abhaychawla deleted the issue-4 branch August 21, 2018 17:10
josehernandezfintecheandomx added a commit to josehernandezfintecheandomx/web-app that referenced this pull request Jul 1, 2023
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

2 participants