Skip to content
This repository has been archived by the owner on Aug 16, 2022. It is now read-only.

feat: Add support for Inspector Classic & Inspector V2 findings (#1305) #1412

Merged
merged 10 commits into from
Aug 12, 2022

Conversation

candiduslynx
Copy link
Contributor

@candiduslynx candiduslynx commented Aug 10, 2022

Summary

Add support for Inspector Classic & Inspector V2 findings (#1305)

Use the following steps to ensure your PR is ready to be reviewed

  • Read the contribution guidelines πŸ§‘β€πŸŽ“
  • Run go fmt to format your code πŸ–Š
  • Lint your changes via golangci-lint run 🚨 (install golangci-lint here)
  • Update or add tests. Learn more about testing here πŸ§ͺ
  • Update the docs by running go run ./docs/docs.go and committing the changes πŸ“ƒ
  • Ensure the status checks below are successful βœ…

@candiduslynx candiduslynx self-assigned this Aug 10, 2022
@candiduslynx candiduslynx requested a review from a team as a code owner August 10, 2022 12:12
@candiduslynx candiduslynx requested review from disq and removed request for a team August 10, 2022 12:12
@candiduslynx candiduslynx linked an issue Aug 10, 2022 that may be closed by this pull request
@candiduslynx candiduslynx added the resource New Resource Request label Aug 10, 2022
.golangci.yml Outdated Show resolved Hide resolved
client/client_test.go Show resolved Hide resolved
resources/provider/provider.go Show resolved Hide resolved
@candiduslynx candiduslynx force-pushed the feature/resource/inspector-findings branch 3 times, most recently from 02dc2a9 to ad2a394 Compare August 11, 2022 03:41
@candiduslynx candiduslynx force-pushed the feature/resource/inspector-findings branch from a47c591 to fd3120c Compare August 12, 2022 14:51
@candiduslynx candiduslynx merged commit 1f1ae96 into main Aug 12, 2022
@candiduslynx candiduslynx deleted the feature/resource/inspector-findings branch August 12, 2022 15:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
resource New Resource Request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AWS Inspector classic and Inspector V2 support
2 participants