-
Notifications
You must be signed in to change notification settings - Fork 56
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
fix: e2e tests #763
Closed
Closed
fix: e2e tests #763
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
…amplify#558) Bumps [react-devtools-core](https://github.com/facebook/react/tree/HEAD/packages/react-devtools-core) from 4.28.0 to 4.28.5. - [Release notes](https://github.com/facebook/react/releases) - [Changelog](https://github.com/facebook/react/blob/main/CHANGELOG.md) - [Commits](https://github.com/facebook/react/commits/HEAD/packages/react-devtools-core) --- updated-dependencies: - dependency-name: react-devtools-core dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* add amplify comment when writing to gitignore * ensure there is space before gitignore content * fix tests
…s/simple_react (aws-amplify#621) * build(deps): bump axios, aws-amplify and @aws-amplify/api Bumps [axios](https://github.com/axios/axios) to 1.6.0 and updates ancestor dependencies [axios](https://github.com/axios/axios), [aws-amplify](https://github.com/aws-amplify/amplify-js) and [@aws-amplify/api](https://github.com/aws-amplify/amplify-js). These dependencies need to be updated together. Updates `axios` from 0.26.0 to 1.6.0 - [Release notes](https://github.com/axios/axios/releases) - [Changelog](https://github.com/axios/axios/blob/v1.x/CHANGELOG.md) - [Commits](axios/axios@v0.26.0...v1.6.0) Updates `aws-amplify` from 5.3.10 to 5.3.12 - [Release notes](https://github.com/aws-amplify/amplify-js/releases) - [Changelog](https://github.com/aws-amplify/amplify-js/blob/main/docs/changelog.md) - [Commits](https://github.com/aws-amplify/amplify-js/compare/aws-amplify@5.3.10...aws-amplify@5.3.12) Updates `@aws-amplify/api` from 5.4.5 to 5.4.6 - [Release notes](https://github.com/aws-amplify/amplify-js/releases) - [Changelog](https://github.com/aws-amplify/amplify-js/blob/main/docs/changelog.md) - [Commits](https://github.com/aws-amplify/amplify-js/compare/@aws-amplify/api@5.4.5...@aws-amplify/api@5.4.6) --- updated-dependencies: - dependency-name: axios dependency-type: indirect - dependency-name: aws-amplify dependency-type: direct:production - dependency-name: "@aws-amplify/api" dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> * chore: update changeset --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: MJ☔ <o.6180339+github@gmail.com>
* test: fix create amplify installation in canary tests * test: fix create amplify installation in canary tests * test: fix create amplify installation in canary tests * test: fix create amplify installation in canary tests
* chore: ping postcss version for dependabot * chore: update changeset
* feat: generate mobile friendly config * feat: generate mobile friendly config * wip * wip * add test * fix that * user agent
* change backend.getStack to backend.createStack with corresponding behavior change * add changeset * fix test assertion
* remove unused type * Create few-steaks-grab.md * Update .changeset/few-steaks-grab.md * lint fix
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* Update profile error message and the profile configure URL --------- Co-authored-by: Hoang Nguyen <hoangaz@amazon.com>
…ws-amplify#650) * feat(formgen): forms should be generated in a nested folder where the ui components are generated
…trategy for tables (aws-amplify#653) * chore: bump to latest data cdk construct, and explicitly set deploy strategy for tables * temp
* fix: client config generation respects sandbox config-out-dir option * fix: client config generation respects sandbox format option * rename sandbox format to config-format * make validateDirectory async * add test for client config options * make validateDirectory private
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* feat: Add usage data metrics for sandbox * update api * add more tests * add changeset * fix automated tests * Refactor packageJsonReader * update tsconfig * add comments * Add usage data emitter factory * Update interface of usage metrics to take metrics and dimensions * is it better? * remove extra variable * changing to 'on' data listener * update snapshots * minor changes * change to use readline * PR updates * Move the uuid types dep * PR updates * update to use __dirname * update snapshots * Replace package json reeader in create-amplify * small rename * remove packageJson.name from installationId
* fix: add required fields to mobile config * more fields * more fields * cs
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…m watching in sandbox (aws-amplify#671) * fix: handling !pattern in gitignore to avoid excluding everything from watching in sandbox * add changeset
…dout (aws-amplify#677) * fix: Force colors to not be stripped off when piping child process stdout * Only pass env variable to child process
* chore: delete examples * chore: changeset
…#744) * Refactor error handling, introduce two new AmplifyErrors * change downstreamerror to cause * lint + api change * does this work? * update comment * Does this work then? * This should work * add cause to super constructor * wrap invokeCDK in try catch * fix tests
* build working * update package lock * updating test cases * update snapshots * update api extract * add changeset * fix api * fix auto-rename mishap * update e2e test case * add additional check to e2e suite * lint fixes * update snapshots * trying this * updating assertion * update snapshots * addressing PR comments * update snapshots * update api extract * update changeset * fix lint * Im dumb * still dumb * remove unused code * hopefully last fix * update snapshots * json parse lambda output
* fix: filter non-sandbox stacks * re-throw unexpected exceptions and use BackendIdentifierConversions to infer type
* construct correct stack name on e2e cleanup * empty changeset
0618
requested review from
edwardfoyle,
sobolk,
Amplifiyer,
swaminator,
renebrandel and
josefaidt
as code owners
December 4, 2023 06:06
🦋 Changeset detectedLatest commit: 96d88d1 The changes in this PR will be included in the next version bump. This PR includes changesets to release 15 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
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.
Issue #, if available:
Description of changes:
tsc
is not supported by PnP Add new moduleResolution option:yarn-pnp
microsoft/TypeScript#28289By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.