#66 Tidy MVP Codebase

2.0.0.0 Standard Feature Product (SFP)
pending
nobody
None
2021-11-21
2020-10-04
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: GVL_ & GC_
  • 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
  • Clones of code should be Functionised or Methodised.

Discussion

  • hermsen

    hermsen - 2021-11-21
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -8,7 +8,8 @@

 * [ ] Delete unused code,
 * [ ] Delete unused variables,
-* [ ] Group all globals / constants in  their respective lists (at most 2 lists?)
+* [ ] Group all globals / constants: GVL_ & GC_ 
 * [ ] 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.
+* [ ] 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
+* [ ] Clones of code should be Functionised or Methodised.
     

  • hermsen

    hermsen - 2021-11-21
    • Description has changed:

    Diff:

    --- old
+++ new
@@ -11,5 +11,5 @@
 * [ ] Group all globals / constants: GVL_ & GC_ 
 * [ ] 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
+* [ ] use the project property [UnitTestingDefine](https://help.codesys.com/webapp/properties;product=LibDevSummary;version=3.5.17.0) to allow unit tests while still hiding irrelevant objects from the user
 * [ ] Clones of code should be Functionised or Methodised.
     

Log in to post a comment.