Ticket #5083 defect closed fixed

Opened 2 years ago

Last modified 2 years ago

Logging docs contain wrong import

Reported by: hynek Owned by:
Priority: normal Milestone:
Component: core Keywords: documentation easy
Cc: oubiwann, thijs, ivank-twisted-bugs@… Branch:
Author: Launchpad Bug:

Description

The example on  http://twistedmatrix.com/documents/current/core/howto/logging.html for DailyLogFile has an invalid import:

from twisted.python.logging import DailyLogFile

It should be:

from twisted.python.logfile import DailyLogFile

Attachments

5083-00-fix-logging-import.patch Download (0.6 KB) - added by ivank 2 years ago.
fix

Change History

1

  Changed 2 years ago by DefaultCC Plugin

  • cc oubiwann added

Changed 2 years ago by ivank

fix

2

  Changed 2 years ago by ivank

  • keywords documentation easy review added
  • cc ivank-twisted-bugs@… added
  • component changed from website to core

3

follow-up: ↓ 4   Changed 2 years ago by jerub

  • keywords review removed

I appreciate the quick turnaround on this. I am willing to commit this patch to trunk if a news file is correctly create. This is a doc fix so if you could amend the patch to contain 5083.doc with a brief statement about what was fixed, that would be best.

I've tested the new code to make sure it's okay, it doesn't do an explicit import of 'log' but considering the context that it is in, I don't think it's important for it to do so.

Please throw in a non-empty news file with a .doc extension, and I will commit your change.

4

in reply to: ↑ 3   Changed 2 years ago by thijs

  • cc thijs added

Replying to jerub:

Please throw in a non-empty news file with a .doc extension, and I will commit your change.

I think this can be merged with an empty .misc file, because it's not major enough (like a new addition to the docs) to be mentioned in the changelog.

5

  Changed 2 years ago by exarkun

  • status changed from new to closed
  • resolution set to fixed

(In [31776]) Apply logging howto example fix to import DailyLogFile from the right module

Author: ivank Reviewer: jerub, thijs, exarkun Fixes: #5083

Change an incorrect import to a correct import in the logging howto.

Note: See TracTickets for help on using tickets.