marking for later reference

no test code added -- not adding duplicate tests; main part is tested in
logic already
Revert "marking for later reference"

This reverts commit 15281f0b3fb0518be75885fa2ce8eb00eb5d4a6e.
Clean up InstructorCourseAddAction class
Mentioned 'GitHub flow' in the process.md
Merge remote-tracking branch 'origin/master' into Issue-1809
Remove redundant comments
Change variable name to be more meaningful
Remove redundant comments to distinguish between course detail and summary
Format and delete comments
Issue 1807
InstructorFeedbackQuestionAddAction
- It is currently possible to add questions with empty question text by modifying the js validator script.(instructorFeedback.js:64)

+ Added assumption to FeedbackAbstractQuestionDetails to assume questionText not empty.
+ Added test for empty questionText, expect assumption faliure.
+ Added test for non-existent enumerations of giver/recipient/question type, expect assumption faliure.
+ Added test for invalid questionNUmber
Remove catching and defining EntityDoesNotExistException since it is never thrown/not possible to be thrown by any method related.
Remove EntityDoesNotExistException since it is already caught by  function createFeedbackQuestion().
Added Assumption.assertNotEmpty/IsEmpty for String. Changed such assertions across TEAMMATES to use the functions.
undo some wrong changes
Added a modifyParamValue() function to make code in test/cases/ui/InstructorFeedbackQuestionAddActionTest.java clearer.
Added a TODO in FeedbackAbstractQuestionDetails to add a validate function.
Add comment about coverage, remove redundant variable
[1809] Clean up InstructorCourseAddAction

fixes #1809
[1813] Clean up InstructorCourseArchiveAction

Fixes #1813
Merge branch 'master' into Issue1806
rename test cases' names