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: 1000! #1000

Merged
merged 6 commits into from
Jul 28, 2020
Merged

docs: 1000! #1000

merged 6 commits into from
Jul 28, 2020

Conversation

lukasheinrich
Copy link
Contributor

@lukasheinrich lukasheinrich commented Jul 27, 2020

Description

This is our #1000's GitHub item!

ReadTheDocs build: https://pyhf.readthedocs.io/en/milestones-1k_github_items/#milestones

Checklist Before Requesting Reviewer

  • Tests are passing
  • "WIP" removed from the title of the pull request
  • Selected an Assignee for the PR to be responsible for the log summary

Before Merging

For the PR Assignees:

  • Summarize commit messages into a comprehensive review of the PR
* 1000 down! :D

@matthewfeickert matthewfeickert added the docs Documentation related label Jul 27, 2020
@matthewfeickert matthewfeickert changed the title 1000! docs: 1000! Jul 27, 2020
README.rst Outdated Show resolved Hide resolved
Co-authored-by: Matthew Feickert <matthew.feickert@cern.ch>
@codecov
Copy link

codecov bot commented Jul 27, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1000   +/-   ##
=======================================
  Coverage   96.65%   96.65%           
=======================================
  Files          59       59           
  Lines        3284     3284           
  Branches      455      455           
=======================================
  Hits         3174     3174           
  Misses         69       69           
  Partials       41       41           
Flag Coverage Δ
#unittests 96.65% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


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 85f130c...c059ecc. Read the comment docs.

99PI reference and also Matthew is bad at remembering syntax
Copy link
Member

@matthewfeickert matthewfeickert left a comment

Choose a reason for hiding this comment

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

LGTM now that I fixed my syntax error. Thanks for the commemorative PR @lukasheinrich, and thanks to you and @kratsg for making this a thing that we do. :)

Copy link
Member

@matthewfeickert matthewfeickert left a comment

Choose a reason for hiding this comment

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

Uh oh, seems that this actually breaks releases to PyPI given how it parses the README

@matthewfeickert matthewfeickert force-pushed the milestones/1k_github_items branch from 41ffff4 to c059ecc Compare July 28, 2020 03:10
@matthewfeickert matthewfeickert merged commit 8a2f7f6 into master Jul 28, 2020
@matthewfeickert matthewfeickert deleted the milestones/1k_github_items branch July 28, 2020 03:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants