Closed Bug 1849963 Opened 3 years ago Closed 29 days ago

sorting_algorithms_comparison.md seems to be unlinked and obsolete

Categories

(Core :: Performance: General, defect)

Product:
Core
Component:
Performance: General
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
157 Branch
Project Flags:
Performance Impact none
Tracking Flags:
Tracking Status
firefox157 --- fixed

People

(Reporter: jstutte, Assigned: v-racine)

References

Details

(Keywords: good-first-bug)

Attachments

(1 file, 1 obsolete file)

Bug 1849963 - Remove obsolete sorting_algorithms_comparison.md from the performance docs. r?jstutte
48 bytes, text/x-phabricator-request
Details | Review

Performance does not link to docs/performance/sorting_algorithms_comparison.md and its content seems to be obsolete.

See Also: → 1842136
Severity: -- → S4

Yeah that seems super obsolete. I think it's part of a profiling tutorial which uses these examples: https://github.com/firefox-devtools/performance-scenarios

I'd say this is superseded by the profiler documentation at https://profiler.firefox.com/docs/ .

Keywords: good-first-bug
Performance Impact: --- → none

New contributor, working on a patch for this bug.

I've submitted a patch removing docs/performance/sorting_algorithms_comparison.md. I confirmed the file is unreferenced across the tree, and ./mach doc builds cleanly without it, with no new warnings.

Are there any tests you'd like me to run before this goes up for review?

Flags: needinfo?(jstutte)
Assignee: nobody → vracine21
Status: NEW → ASSIGNED
Attachment #9632904 - Attachment is obsolete: true
Pushed by jstutte@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/972d12883106 https://hg.mozilla.org/integration/autoland/rev/857cbb45f6dd Remove obsolete sorting_algorithms_comparison.md from the performance docs. r=jstutte

https://hg.mozilla.org/mozilla-central/rev/857cbb45f6dd

Status: ASSIGNED → RESOLVED
Closed: 29 days ago
status-firefox157: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 157 Branch
Flags: needinfo?(jstutte)
You need to log in before you can comment on or make changes to this bug.