Skip to content

[requires.io] dependency update on master branch - #123

Merged
wsanchez merged 71 commits into
masterfrom
requires-io-master
Aug 28, 2020
Merged

wsanchez merged 71 commits into
masterfrom
requires-io-master

Conversation

@glyph

@glyph glyph commented Apr 5, 2020

Copy link
Copy Markdown
Collaborator

No description provided.

@codecov-io

codecov-io commented Apr 5, 2020 •

Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #123   +/-   ##
=======================================
  Coverage   98.67%   98.67%           
=======================================
  Files          13       13           
  Lines        1809     1809           
  Branches      209      209           
=======================================
  Hits         1785     1785           
  Misses         12       12           
  Partials       12       12           

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 79e4065...dbac761. Read the comment docs.

@mahmoud mahmoud mentioned this pull request Apr 5, 2020
@glyph
glyph requested a review from mahmoud April 6, 2020 03:11
@glyph

glyph commented Apr 6, 2020

Copy link
Copy Markdown
Collaborator Author

@mahmoud I can't review this because "I" submitted it, so please rubber stamp

@wsanchez

wsanchez commented Apr 6, 2020

Copy link
Copy Markdown
Contributor

This includes stuff I'm not sure it should.

wsanchez added 2 commits April 6, 2020 12:34
* master:
  Couple more cleanup bits

# Conflicts:
#	tox.ini
@wsanchez

wsanchez commented Apr 6, 2020

Copy link
Copy Markdown
Contributor

@mahmoud @glyph This is not all that rubber-stampy, it turns out.

I've removed type info from the docstrings, as Sphinx now includes them in the function signatures.

Comment thread tox.ini Outdated
skip_install = True

deps =
coverage==4.5.4 # rq.filter: <5 # coverage 5.0 drops Python 3.4 support

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

rq.filter doesn't seem to be working

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

You have to commit it to master before it takes effect. Committing to this branch doesn't do anything.

@wsanchez wsanchez Apr 14, 2020 •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

…12 days ago. Lemme reverting that again.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I can't figure out what we need to do to tell requires.io to stop updating this. I must have the syntax wrong or something… @glyph ?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I don't see comments-in-the-comment in any other projects I've done with requires.io, so maybe rather than:

    coverage==4.5.4  # rq.filter: <5  # coverage 5.0 drops Python 3.4 support

do

    # coverage 5.0 drops Python 3.4 support
    coverage==4.5.4  # rq.filter: <5

And then close this PR to see what it does when it reopens it.

@wsanchez

Copy link
Copy Markdown
Contributor

@mahmoud or @glyph If you can thumbs up the docs change, I'll fix the coverage version up prior to merging (requires.io will keep stomping the fix in the meantime).

@wsanchez

Copy link
Copy Markdown
Contributor

This PR is getting pretty wooly due to the Sphinx update… how about I break it into separate PR steps:

  1. Update requirements, and restrict Sphinx to <3.
  2. Update Sphinx and fix typing
  3. Zero this PR back out.

@glyph

glyph commented Jun 24, 2020

Copy link
Copy Markdown
Collaborator Author

Please do!

@wsanchez
wsanchez self-requested a review August 28, 2020 20:39
@wsanchez

Copy link
Copy Markdown
Contributor

OK looks like this is all good now. Merging.

@wsanchez
wsanchez merged commit 7d2b0be into master Aug 28, 2020
@wsanchez
wsanchez deleted the requires-io-master branch August 28, 2020 20:39
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.

5 participants