Skip to content

Add linting and specify missing version constraints - #28

Merged
htunnicliff merged 5 commits into
masterfrom
htunnicliff/linting
Jun 18, 2026
Merged

Add linting and specify missing version constraints#28
htunnicliff merged 5 commits into
masterfrom
htunnicliff/linting

Conversation

@htunnicliff

@htunnicliff htunnicliff commented Jun 11, 2026

Copy link
Copy Markdown
Member

Changes

  • Breaking: Remove default lambda runtime value
  • Breaking: Add missing version constraints
  • Add tflint
  • Add CI workflow for linting
  • Add missing types to variables

Notes

The version constraints added are for the current provider versions at the time the source code was added to this repo, not the present-day current provider versions. This should not be breaking, but I've noted it as possibly breaking in the changelog.

@htunnicliff htunnicliff self-assigned this Jun 11, 2026
@htunnicliff
htunnicliff force-pushed the htunnicliff/linting branch from f84405c to f146192 Compare June 11, 2026 20:09
@htunnicliff
htunnicliff force-pushed the htunnicliff/linting branch from f146192 to ae5e518 Compare June 11, 2026 20:14
@htunnicliff
htunnicliff marked this pull request as ready for review June 17, 2026 22:34
@htunnicliff htunnicliff changed the title Install tflint Add linting and specify missing version constraints Jun 17, 2026
@htunnicliff
htunnicliff requested a review from a team June 17, 2026 22:35
@htunnicliff
htunnicliff force-pushed the htunnicliff/linting branch from 710420d to 8c482ae Compare June 17, 2026 22:37
@htunnicliff
htunnicliff merged commit 0e20521 into master Jun 18, 2026
1 check passed
@htunnicliff
htunnicliff deleted the htunnicliff/linting branch June 18, 2026 00:43
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.

2 participants