Differential D12900

Remove file TODOPORT
ClosedPublic
Actions

Authored by gregormi on May 15 2018, 5:12 PM.

Details

Reviewers
None
Group Reviewers
KDE Games
Commits
R417:6ba2528a64c3: Remove file TODOPORT
Summary

This is the latest status of the items on the now deleted list:

  • 2007: SOMETIMES BUTTONS IN MAIN DIALOG MAKE KSUDOKU CRASH
    • 2018: Is this still an issue?
  • 2007: export system
    • 2018: What is meant by that?
  • 2007: status bar text disappear sometimes
    • 2018: Is this still an issue?
  • 2007: flickering
    • 2018: Is this still an issue?
  • 2007: knewstuff crashes
    • 2018: Where is this used?
  • 2007: KConfig
    • 2018: Is this still an issue?

Diff Detail

Repository
R417 KSudoku
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
gregormi created this revision.May 15 2018, 5:12 PM
Restricted Application added a reviewer: KDE Games. · View Herald TranscriptMay 15 2018, 5:12 PM
Restricted Application added a subscriber: kde-games-devel. · View Herald Transcript
gregormi requested review of this revision.May 15 2018, 5:12 PM
aacid added a subscriber: aacid.May 15 2018, 6:01 PM

I'd just remove the todo list, maintaining a todo list in code is never a good idea, that's what bug trackers and task tracking in phabricator is for IMHO

gregormi updated this revision to Diff 34230.May 15 2018, 8:39 PM
  • Remove TODOPORT
gregormi retitled this revision from comments on todo list to Remove TODOPORT.May 15 2018, 8:41 PM
gregormi edited the summary of this revision. (Show Details)
gregormi retitled this revision from Remove TODOPORT to Remove file TODOPORT.
aacid added a comment.May 15 2018, 9:08 PM

I'd say if noone disagrees in a week or so feel free to remove it

This revision was not accepted when it landed; it landed in state Needs Review.May 23 2018, 7:15 AM
Closed by commit R417:6ba2528a64c3: Remove file TODOPORT (authored by gregormi). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
DMTODOPORT (6 lines)

Diff 34686

View Options

TODOPORT

Loading...