diff --git a/.devcontainer/devcontainer.json b/.devcontainer/devcontainer.json index be6a9a17..c4d41376 100755 --- a/.devcontainer/devcontainer.json +++ b/.devcontainer/devcontainer.json @@ -166,21 +166,48 @@ "doxdocgen.generic.useGitUserName": true, // Todo Tree Extention Settings "todo-tree.general.tags": [ - "FIXME:", - "TODO:", - "LEAD:", - "ISSUE NOTE:" + "TODO", + "BUG", + "HACK", + "FIXME", + "LEAD", + "ISSUE NOTE", + "TEST", + "[ ]", + "[x]" ], + "todo-tree.highlights.defaultHighlight": { + "icon": "alert", + "type": "text", + "foreground": "#00ff00", + "background": "#ffffff", + "opacity": 50, + "iconColour": "#00ff00" + }, "todo-tree.highlights.customHighlight": { "TODO": { - "icon": "check", + "icon": "bookmark", "type": "line", "iconColour": "#d000ff", "foreground": "#ffffff", "background": "#d000ff" }, + "BUG": { + "icon": "bug", + "type": "line", + "iconColour": "#ff8c00", + "foreground": "#ffffff", + "background": "#ff8c00" + }, + "HACK": { + "icon": "circle-slash", + "type": "line", + "iconColour": "#ff1e00", + "foreground": "#ffffff", + "background": "#ff1e00" + }, "FIXME": { - "icon": "beaker", + "icon": "alert-fill", "type": "line", "iconColour": "#ff0000", "foreground": "#ffffff", @@ -189,22 +216,22 @@ "LEAD": { "icon": "person", "type": "line", - "iconColour": "#ffffff", - "foreground": "#000000", - "background": "#ffffff" + "iconColour": "#0700d8", + "foreground": "#ffffff", + "background": "#0700d8" }, "ISSUE NOTE": { "icon": "book", "type": "line", "iconColour": "#808080", - "foreground": "#000000", + "foreground": "#ffffff", "background": "#808080" }, "TEST": { - "icon": "book", + "icon": "beaker", "type": "line", "iconColour": "#c5cb04", - "foreground": "#000000", + "foreground": "#ffffff", "background": "#ccd514" } }, diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 0c891b24..f0f01295 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -53,6 +53,21 @@ Commit names should be no more than 50 characters. They should have `#` to mark that the pull request will fix an issue. +## Marking Issues in Code + +### Utilizing TODO-Trees + +A list of action items can be built in the TODO extension tab using these keywords to highlight and mark sections of code for later review: +- // TODO: Implement the behavior specific to the Approaching Object state +- // BUG: This breaks occasionally. +- // HACK: This is bad, never do this. +- // FIXME: Broken +- // LEAD: Check this out... +- // ISSUE NOTE: This is messed up. +- // TEST: Make sure this works, write unit tests. +- // [ ] This is a list item. +- // [x] This is a checked list item. + ## Styleguides ### Variable Names diff --git a/src/states/ApproachingObjectState.cpp b/src/states/ApproachingObjectState.cpp index c1d2ba8f..4f97cf60 100644 --- a/src/states/ApproachingObjectState.cpp +++ b/src/states/ApproachingObjectState.cpp @@ -77,7 +77,7 @@ namespace statemachine ******************************************************************************/ void ApproachingObjectState::Run() { - // TODO: Implement the behavior specific to the Approaching Object state + // TODO: Implement the behavior specific to the Approaching Object state. LOG_DEBUG(logging::g_qSharedLogger, "ApproachingObjectState: Running state-specific behavior."); }