Skip to content

feat: add new aws iam identity center scripts#24

Merged
dannysteenman merged 2 commits intomainfrom
feat/sso-scripts
Mar 16, 2023
Merged

feat: add new aws iam identity center scripts#24
dannysteenman merged 2 commits intomainfrom
feat/sso-scripts

Conversation

@dannysteenman
Copy link
Copy Markdown
Member

@dannysteenman dannysteenman commented Mar 16, 2023

Pull request checklist

Please check if your PR fulfills the following requirements:

  • Docstrings or comments have been reviewed and added / updated if needed
  • The change has been tested and confirmed working (Tested on? OS, AWS CLI Version)

Pull request type

Please check the type of change your PR introduces:

  • Bugfix (if it's an open issue, please add the issue number):
  • Feature e.g. new script (please describe):
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

Other information

@dannysteenman dannysteenman merged commit 75612c5 into main Mar 16, 2023
@dannysteenman dannysteenman deleted the feat/sso-scripts branch March 16, 2023 13:45
dannysteenman added a commit that referenced this pull request Mar 19, 2026
* feat(cli): split live discovery into discover command

* feat(sdk): route live discovery through resource explorer

* docs: align discovery guidance with resource explorer model

* fix(cli): preserve discover policy exit code

* refactor(cli): return discover exit codes explicitly

* test(sdk): prove lambda client reuse under concurrency

* test(sdk): isolate aws region provider chain test

* refactor: remove legacy discovery compatibility

* test(rules): enforce discovery metadata invariant

* refactor(sdk): rename scanner facade to client

* fix(sdk): merge runtime config onto loaded config

* fix(discovery): validate region input and sanitize errors

* fix(cli): sanitize unknown runtime errors

* fix(cli): restore scan coverage and share validators

* fix(cli): ignore parent all region for init
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.

1 participant