Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Include a default configuration file in the 'docs' directory. #4791

Merged
merged 3 commits into from
Mar 4, 2019

Conversation

richvdh
Copy link
Member

@richvdh richvdh commented Mar 4, 2019

This works by simply checking a file into git, and adding a CI job which will remind us to update it if we forget.

Fixes #1433.

@codecov
Copy link

codecov bot commented Mar 4, 2019

Codecov Report

Merging #4791 into develop will decrease coverage by 0.1%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #4791      +/-   ##
===========================================
- Coverage    75.13%   75.02%   -0.11%     
===========================================
  Files          340      340              
  Lines        34918    35134     +216     
  Branches      5722     5797      +75     
===========================================
+ Hits         26234    26358     +124     
- Misses        7072     7152      +80     
- Partials      1612     1624      +12

@richvdh richvdh requested a review from a team March 4, 2019 14:00
Copy link
Member

@erikjohnston erikjohnston left a comment

Choose a reason for hiding this comment

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

Otherwise lgtm

plugins:
- docker#v3.0.1:
image: "python:3.6"

Copy link
Member

Choose a reason for hiding this comment

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

This seemed to take over 1 minute, so I'd be tempted to move after the wait

Copy link
Member Author

Choose a reason for hiding this comment

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

yeah, it's downloading and installing all of the python deps. It seems a shame that (a) we can't do better caching here, and (b) the virtualenv isn't shared between the builds.

@richvdh richvdh merged commit 8e28bc5 into develop Mar 4, 2019
@richvdh richvdh deleted the rav/config_doc branch March 4, 2019 17:15
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.

2 participants