Skip to content

Suppress error messages in clear_logs.bat#137

Merged
KathrynBaker merged 2 commits into
masterfrom
FreddieAkeroyd-patch-1
Jun 19, 2026
Merged

Suppress error messages in clear_logs.bat#137
KathrynBaker merged 2 commits into
masterfrom
FreddieAkeroyd-patch-1

Conversation

@FreddieAkeroyd

@FreddieAkeroyd FreddieAkeroyd commented Jun 19, 2026

Copy link
Copy Markdown
Member

These access denied message do not usually indicate a problem and look confusing in log when there is a real problem, maybe anti-virus related?


Code Review

  • Is the code of an acceptable quality?
  • Do the changes function as described and is it robust?
  • Have the changes been documented in the release notes?

Final Steps

  • Reviewer has moved the release notes entry for this ticket in the "Changes merged into master" section

Redirect error messages to null while deleting files.
Updated comment to clarify potential antivirus issues.
@FreddieAkeroyd FreddieAkeroyd moved this to Flash Review in Tasks Jun 19, 2026
@KathrynBaker KathrynBaker merged commit aac3b7b into master Jun 19, 2026
4 checks passed
@KathrynBaker KathrynBaker deleted the FreddieAkeroyd-patch-1 branch June 19, 2026 16:28
@KathrynBaker KathrynBaker removed this from Tasks Jun 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants