#66 Tidy MVP Codebase

pending
nobody
None
2020-10-18
2020-10-04
h-hermsen
No
  • Add verbose in 4 levels:
    • None
    • Error
    • Error, Warning
    • Error, Warning, Info

After we have realised a full mvp with unit tests:

  • Delete unused code,
  • Delete unused variables,
  • Group all globals / constants in their respective lists (at most 2 lists?)
  • Add/revise explanative comment headers to all functions/function blocks/programs etc etc.
  • Remove all {...} by solving them.
  • use the project property UnitTestingDefine to allow unit tests while still hiding irrelevant objects from the user.

Discussion

  • h-hermsen

    h-hermsen - 2020-10-04
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -2,4 +2,5 @@
    
     [ ] Delete unused code,
     [ ] Delete unused variables,
    +[ ] Add/revise explanative comment headers to all functions/function blocks/programs etc etc.
     [ ] Remove all {...} by solving them.
    
     
  • h-hermsen

    h-hermsen - 2020-10-04
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -2,5 +2,6 @@
    
     [ ] Delete unused code,
     [ ] Delete unused variables,
    +[ ] Group all globals / constants in  their respective lists (at most 2 lists?)
     [ ] Add/revise explanative comment headers to all functions/function blocks/programs etc etc.
     [ ] Remove all {...} by solving them.
    
     
  • h-hermsen

    h-hermsen - 2020-10-05
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -1,7 +1,13 @@
     After we have realised a full mvp with unit tests:
    
    -[ ] Delete unused code,
    -[ ] Delete unused variables,
    -[ ] Group all globals / constants in  their respective lists (at most 2 lists?)
    -[ ] Add/revise explanative comment headers to all functions/function blocks/programs etc etc.
    -[ ] Remove all {...} by solving them.
    +* [ ] Delete unused code,
    +* [ ] Delete unused variables,
    +* [ ] Group all globals / constants in  their respective lists (at most 2 lists?)
    +* [ ] Add/revise explanative comment headers to all functions/function blocks/programs etc etc.
    +* [ ] Remove all {...} by solving them.
    +* [ ] Add verbose in 4 levels:
    +    * [ ] None
    +    * [ ] Error
    +    * [ ] Error, Warning
    +    * [ ] Error, Warning, Info
    +* [ ] Add an Enable (True, False) signal to the EoN
    
     
  • h-hermsen

    h-hermsen - 2020-10-05
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -1,3 +1,10 @@
    +* [ ] Add verbose in 4 levels:
    +    * [ ] None
    +    * [ ] Error
    +    * [ ] Error, Warning
    +    * [ ] Error, Warning, Info
    +* [ ] Add an Enable (True, False) signal to the EoN
    +
     After we have realised a full mvp with unit tests:
    
     * [ ] Delete unused code,
    @@ -5,9 +12,3 @@
     * [ ] Group all globals / constants in  their respective lists (at most 2 lists?)
     * [ ] Add/revise explanative comment headers to all functions/function blocks/programs etc etc.
     * [ ] Remove all {...} by solving them.
    -* [ ] Add verbose in 4 levels:
    -    * [ ] None
    -    * [ ] Error
    -    * [ ] Error, Warning
    -    * [ ] Error, Warning, Info
    -* [ ] Add an Enable (True, False) signal to the EoN
    
     
  • i-campbell

    i-campbell - 2020-10-06
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -12,3 +12,4 @@
     * [ ] Group all globals / constants in  their respective lists (at most 2 lists?)
     * [ ] Add/revise explanative comment headers to all functions/function blocks/programs etc etc.
     * [ ] Remove all {...} by solving them.
    +* [ ] use the project property [UnitTestingDefine](https://help.codesys.com/webapp/properties;product=LibDevSummary;version=3.5.15.0) to allow unit tests while still hiding irrelevant objects from the user.
    
     
  • h-hermsen

    h-hermsen - 2020-10-09
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -1,9 +1,9 @@
    -* [ ] Add verbose in 4 levels:
    -    * [ ] None
    -    * [ ] Error
    -    * [ ] Error, Warning
    -    * [ ] Error, Warning, Info
    -* [ ] Add an Enable (True, False) signal to the EoN
    +* [x] Add verbose in 4 levels:
    +    * [x] None
    +    * [x] Error
    +    * [x] Error, Warning
    +    * [x] Error, Warning, Info
    +* [x] Add an Enable (True, False) signal to the EoN
    
     After we have realised a full mvp with unit tests:
    
     
  • h-hermsen

    h-hermsen - 2020-10-13
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -3,7 +3,6 @@
         * [x] Error
         * [x] Error, Warning
         * [x] Error, Warning, Info
    -* [x] Add an Enable (True, False) signal to the EoN
    
     After we have realised a full mvp with unit tests:
    
     
  • h-hermsen

    h-hermsen - 2020-10-18
    • status: open --> pending
    • Milestone: 1.0.0.0 Minimum Viable Product (MVP) --> 2.0.0.0 Standard Feature Product (SFP)
     
  • h-hermsen

    h-hermsen - 2020-10-18

    Will be solved further along v2.0.0.0

     

Log in to post a comment.