Skip to content

improved duplicate code in error handling#124

Merged
satyakigh merged 1 commit intomainfrom
fix/duplicate-code/error-handling
Oct 23, 2025
Merged

improved duplicate code in error handling#124
satyakigh merged 1 commit intomainfrom
fix/duplicate-code/error-handling

Conversation

@tylerymarkowitz
Copy link
Copy Markdown
Contributor

@tylerymarkowitz tylerymarkowitz commented Oct 23, 2025

  • Code duplication found from npm run check:duplicates
  • Extract common cleanup error logging utility in StackActionOperations
  • Extract common service error logging pattern in CfnLintService
  • Extract common code action error logging pattern in CodeActionService
  • No new logic or behavior was added

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@tylerymarkowitz tylerymarkowitz force-pushed the fix/duplicate-code/error-handling branch from b4880b5 to 9858745 Compare October 23, 2025 20:29
@tylerymarkowitz tylerymarkowitz force-pushed the fix/duplicate-code/error-handling branch from 9858745 to 17708a8 Compare October 23, 2025 20:29
@tylerymarkowitz tylerymarkowitz marked this pull request as ready for review October 23, 2025 20:34
@tylerymarkowitz tylerymarkowitz requested a review from a team as a code owner October 23, 2025 20:34
@satyakigh satyakigh merged commit aa6d00b into main Oct 23, 2025
8 checks passed
@satyakigh satyakigh deleted the fix/duplicate-code/error-handling branch November 4, 2025 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants