doc8

ignore-path-errors uses absolute path

Bug #1502391 reported by Frazer McLean on 2015-10-03

This bug affects 3 people

Affects		Status	Importance	Assigned to	Milestone
	doc8	New	Undecided	Unassigned

Function `parse_ignore_path_errors` in doc8/main.py does this:

path = os.path.abspath(path.strip())

So, in the `validate` function, `ignore_targeted` is a dictionary of absolute paths.

Then, the following line uses `f.filename`, which is a relative path.

targeted_ignoreables = set(ignore_targeted.get(f.filename, set()))

Thus, the lookup doesn't find the correct path.

Antoine Cabot (antoinecabot) on 2016-03-14

Changed in watcher:
status:	New → Triaged

Andy Hsiang (yh418t) on 2016-05-16

Changed in watcher:
assignee:	nobody → Andy Hsiang (yh418t)

Andy Hsiang (yh418t) on 2016-05-16

Changed in doc8:
assignee:	nobody → Andy Hsiang (yh418t)

Andy Hsiang (yh418t) on 2016-05-24

Antoine Cabot (antoinecabot) on 2016-07-08

Revision history for this message

Brecht Machiels (brechtm) wrote on 2016-08-05:

This makes this option useless. Any plans on fixing this and releasing a new version? Thanks.

Antoine Cabot (antoinecabot) on 2016-08-17

Changed in watcher:
milestone:	newton-rc1 → newton-3

David TARDIVEL (david-tardivel) on 2016-08-17

Changed in watcher:
assignee:	nobody → David TARDIVEL (david-tardivel)
Changed in doc8:
assignee:	nobody → David TARDIVEL (david-tardivel)
assignee:	David TARDIVEL (david-tardivel) → nobody

David TARDIVEL (david-tardivel) on 2016-08-17

no longer affects:

watcher

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Bug watches keep track of this bug in other bug trackers.