Conversation
| sonar.host.url=https://sonarcloud.io | ||
| sonar.python.version=3.11 | ||
| sonar.exclusions=**/e2e/**,**/e2e_batch/**,**/temporary_sandbox/**,**/devtools/**,**/proxies/**,**/scripts/**,**/terraform/**,**/tests/** | ||
| sonar.exclusions=**/e2e/**,**/e2e_batch/**,**/temporary_sandbox/**,**/devtools/**,**/proxies/**,**/scripts/**,**/terraform/**,**/tests/**,redis_sync/src/log_decorator.py |
There was a problem hiding this comment.
I think there's probably a way to mark an error as fine to ignore in the Sonar UI itself. Might be better if I do that so we don't forget to remove this exclusion later on
There was a problem hiding this comment.
Trying the #NOSONAR tag
There was a problem hiding this comment.
#NOSONAR tag seems to be an AI hallucination. Search of t'internet suggests it is not possible to suppress locally. So, I have added a TODO to the comment that it can be removed when the code is relocated for reuse
There was a problem hiding this comment.
Ah sorry, I meant if you remove this exclusion I can probably go into the Sonar UI and ignore it there, no code change needed.
nhsdevws
force-pushed
the
VED-369-Redis-Sync-Firehose-Logging
branch
from
59f6d46 to
30b474e
Compare
|
2 participants
Summary
VED-369 Config upload Splunk Firehose
Description Add logging to firehose to the redis-sync lambda
****
Notes:
Reviews Required
Review Checklist
ℹ️ This section is to be filled in by the reviewer.