Skip to content

fix: resolve cfn-lint findings in cloudformation.json#393

Merged
credfeto merged 2 commits into
mainfrom
chore/pre-commit-cleanup
Jul 13, 2026
Merged

fix: resolve cfn-lint findings in cloudformation.json#393
credfeto merged 2 commits into
mainfrom
chore/pre-commit-cleanup

Conversation

@dnyw4l3n13

Copy link
Copy Markdown
Collaborator

Description

How Has This Been Tested

  • All unit tests pass.
  • All integration tests pass.
  • Manual Testing:

Types of changes

  • Docs change
  • Refactoring
  • Dependency upgrade
  • Additional Unit Tests\Integration Tests
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing
    functionality to change)
  • Removed no-longer used code

Deployment Configuration Changes

  • Requires deployment configuration changes as specified below and in CHANGELOG.md

Checklist

  • I have added tests to cover my changes.
  • Unreleased section of CHANGELOG.md has been updated with details of this PR.

credfeto added 2 commits July 13, 2026 10:44
… heading punctuation)

Cleans up files flagged by the pre-commit baseline check: missing
final newlines, a stray trailing space in the Dockerfile, and a
markdownlint MD026 violation in the README heading.
Remove the unused AwsContainerRegistry parameter, add explicit
DeletionPolicy/UpdateReplacePolicy to LogGroup (preserving current
Delete behaviour), and drop DependsOn entries already implied by Ref,
plus trim trailing whitespace so the pre-commit baseline is clean.

Prompt: run pre-commit-check and fix any issues identified until clean and then push to a new branch
@dnyw4l3n13 dnyw4l3n13 added the auto-pr Pull request created automatically label Jul 13, 2026
@credfeto credfeto marked this pull request as ready for review July 13, 2026 10:57
@credfeto credfeto requested a review from a team as a code owner July 13, 2026 10:57
Copilot AI review requested due to automatic review settings July 13, 2026 10:57
@credfeto credfeto merged commit 48601bd into main Jul 13, 2026
33 checks passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@credfeto credfeto deleted the chore/pre-commit-cleanup branch July 13, 2026 10:57
@credfeto

Copy link
Copy Markdown
Member

Super-linter summary

Language Validation result

All files and directories linted successfully

For more information, see the GitHub Actions workflow run

Powered by Super-linter

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-pr Pull request created automatically

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants