-
Notifications
You must be signed in to change notification settings - Fork 79
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: merge main into release #3081
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* bump cross-spawn version in scripts/yarn.lock to patched version * remove redundant cross-spawn version
…ormerV2 E2E test (#3041)
* log config * add: logConfig main logic * add: test for logConfig * update: API.md * update: clean up code, update docs, add test, add reimport and reexport * update: redefine LogConfig * update: docs and export Logging * update: docs * Update packages/amplify-graphql-api-construct/src/types.ts Co-authored-by: Tim Schmelter <[email protected]> * Update packages/amplify-graphql-api-construct/src/types.ts Co-authored-by: Tim Schmelter <[email protected]> * Update packages/amplify-graphql-api-construct/src/types.ts Co-authored-by: Tim Schmelter <[email protected]> * update: docs and specify default values * chore: prettier * break circular dependency in setupLogConfig with scoped down policy (#2938) * add: code that creates circular dependency * add: create standalone policy and attach it to break dependency cycle * update: comments * remove ampliy owned log group logic, keep to appsync owned log group with managed policy * added retention day logic, merged main * finalized PR, extracted api and dependency licenses * added e2e tests for log config * chore for PR * added e2e to workflow * replicate unit tests in transformer-core * dropped api-construct package coverage from 90 to 89.5 * updated dependency licenses * removed redundent import * fix: linting errors * chore: resolved comments * Update packages/amplify-graphql-api-construct-tests/src/__tests__/log-config.test.ts Co-authored-by: Peter V. <[email protected]> * fix: addressed comments * chore: prettier * fix: use SDK v3 for verifyLogConfig * merged main * fix: verifyLogsWithRequestId before verifyLogConfig * chore: lint, extract-api * debug: added log statement for region * fix: specify the region for client * chore: standardize sdk version to 3.624.0 * fix: change abort-controller version * fix: streamlined e2e and unit tests * chore: lint --------- Co-authored-by: Tim Schmelter <[email protected]> Co-authored-by: Peter V. <[email protected]>
Bumps [nanoid](https://github.com/ai/nanoid) from 3.3.1 to 3.3.8. - [Release notes](https://github.com/ai/nanoid/releases) - [Changelog](https://github.com/ai/nanoid/blob/main/CHANGELOG.md) - [Commits](ai/nanoid@3.3.1...3.3.8) --- updated-dependencies: - dependency-name: nanoid dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
The information in contained in dependency_licenses.txt
* chore: construct dependencies fix Signed-off-by: Kevin Shan <[email protected]> * chore: verification scripts update + remove non-runtime bundled packages Signed-off-by: Kevin Shan <[email protected]> * chore: root package.json fix + dependency sort Signed-off-by: Kevin Shan <[email protected]> * fix: bump awd-cde-lib version to resolve dependency conflict with @aws-amplify/auth-construct Signed-off-by: Kevin Shan <[email protected]> * chore: yarn.lock build Signed-off-by: Kevin Shan <[email protected]> * fix: bump aws-cdk lib version to 2.168.0 Signed-off-by: Kevin Shan <[email protected]> * chore: move deps verification changes to separate PR * fix: cdk dep fix * chore: update yarn.lock * chore: test dep fix * chore: test deps fix * fix: cdk aws-sdk bundling fix --------- Signed-off-by: Kevin Shan <[email protected]>
palpatim
approved these changes
Dec 17, 2024
dpilch
approved these changes
Dec 17, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of changes
CDK / CloudFormation Parameters Changed
Issue #, if available
Description of how you validated changes
Checklist
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.