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

Documentation links fix - callInThread and callFromThread were 404'd #703

Merged

Conversation

jisaacstone
Copy link
Contributor

@jisaacstone jisaacstone commented Feb 3, 2017

https://twistedmatrix.com/trac/ticket/9027

IReactorThreads.callInThread -> IReactorInThreads.callInThread
IReactorThreads.callFromThread -> IReactorFromThreads.callFromThread

Live Broken Links Found Here:
http://twistedmatrix.com/documents/current/core/howto/logger.html
http://twistedmatrix.com/documents/current/core/howto/threading.html

@codecov-io
Copy link

codecov-io commented Feb 3, 2017

Codecov Report

Merging #703 into trunk will decrease coverage by -1.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            trunk     #703      +/-   ##
==========================================
- Coverage   91.27%   90.19%   -1.09%     
==========================================
  Files         844      844              
  Lines      147472   147472              
  Branches    13047    13047              
==========================================
- Hits       134612   133013    -1599     
- Misses      10623    12186    +1563     
- Partials     2237     2273      +36

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 78679af...4571ca3. Read the comment docs.

IReactorThreads.callInThread -> IReactorInThreads.callInThread
IReactorThreads.callFromThread -> IReactorFromThreads.callFromThread
@jisaacstone jisaacstone force-pushed the 9027-jisaacstone-broken_ireactorthreads_links branch from a8f45ea to e5215f9 Compare February 3, 2017 17:07
@jisaacstone
Copy link
Contributor Author

ping

@glyph
Copy link
Member

glyph commented Feb 14, 2017

Pong :).

If you want to accelerate your review, one way you can do that is by providing your own review on one of the reviews in the queue. The formatting on that page should show you which tickets are open to review by non-committers.

Thanks for your contribution!

Copy link
Member

@glyph glyph left a comment

Choose a reason for hiding this comment

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

Thanks for this contribution! We should probably add some link-checking automation to our CI system at some point :).

This seems to be a good fix, and the coverage changes are spurious.

@glyph glyph merged commit ba90824 into twisted:trunk Feb 17, 2017
@jisaacstone jisaacstone deleted the 9027-jisaacstone-broken_ireactorthreads_links branch February 17, 2017 15:52
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.

4 participants