search

Spyderisk / domain-csv2nq

Convert from domain model CSV source files into NQ artifact
Apache License 2.0
0 stars 0 forks source link

Extended command line option '-u' so it negates construction state flags #5

Closed mike1813 closed 4 months ago

mike1813 commented 11 months ago

in addition to MS/TWA visibility flags, addressing #4.

Useful for domain model development, when it is often necessary to inspect what happened during the construction pattern processing phase of validation. Deleting assets and links that are only needed during construction makes this very difficult, so having the option to retain them makes sense.

sonarcloud[bot] commented 11 months ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

No Coverage information No Coverage information
0.0% 0.0% Duplication

mike1813 commented 11 months ago

@scp93ch : sonarlint says the code is too smelly.

All I did was add an 'and not unfiltered' clauses to a couple of logical expressions used for conditional branches. I don't doubt we could 'refactor' to make it less smelly, but to do that would require significant code restructuring so I am not bothering with this.