Skip to content
This repository has been archived by the owner on Nov 21, 2023. It is now read-only.

gh-pages fixes #240

Merged
merged 2 commits into from
Nov 6, 2018
Merged

gh-pages fixes #240

merged 2 commits into from
Nov 6, 2018

Conversation

jklukas
Copy link
Contributor

@jklukas jklukas commented Nov 5, 2018

Just a few more changes to fix observed problems with docs and CI

@jklukas jklukas requested a review from wlach November 5, 2018 17:43
@@ -111,7 +111,7 @@ jobs:
git config user.email "fx-data-platform@mozilla.com"
git config user.name "CircleCI docs-deploy job"
npm install -g --silent gh-pages@2.0.1
gh-pages -d docs/_build/html
gh-pages --dotfiles --message "[skip ci] Updates" --dist docs/_build/html
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The --dotfiles flag is necessary for the .nojekyll file to get committed. The [skip ci] message prevents CircleCI from trying to build the gh-pages branch after this commit (which we expect to fail because it doesn't have the source code).

tox.ini Outdated
# TODO: Remove this once moto restores compatibility
deps =
boto3==1.7.84

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The moto issue is now fixed, so we can remove this restriction.

@jklukas jklukas removed the request for review from wlach November 5, 2018 17:46
@jklukas
Copy link
Contributor Author

jklukas commented Nov 5, 2018

Spoke too soon on the moto fix. I'm investigating now.

@codecov-io
Copy link

codecov-io commented Nov 5, 2018

Codecov Report

Merging #240 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #240   +/-   ##
=======================================
  Coverage   80.05%   80.05%           
=======================================
  Files          11       11           
  Lines        1053     1053           
=======================================
  Hits          843      843           
  Misses        210      210
Flag Coverage Δ
#py27 79.86% <ø> (ø) ⬆️
#py35 79.01% <ø> (ø) ⬆️
#py36 79.01% <ø> (ø) ⬆️
Impacted Files Coverage Δ
moztelemetry/spark.py 81.25% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 679b89b...c3eb2c3. Read the comment docs.

boto3==1.7.84
setenv =
AWS_SECRET_ACCESS_KEY=dummy_secret_needed_for_moto_tests
AWS_ACCESS_KEY_ID=dummy_key_needed_for_moto_tests
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Follows new example in getmoto/moto#1907

Copy link
Contributor

@wlach wlach left a comment

Choose a reason for hiding this comment

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

ta!

@jklukas jklukas merged commit fb68074 into master Nov 6, 2018
@jklukas jklukas deleted the ghpages-docs branch November 6, 2018 14:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants