Skip to content
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

Add import mappings to non-dynamic, and correct EscapeHatchProps import type. #277

Merged
merged 3 commits into from
Dec 6, 2021
Merged

Conversation

aherschel
Copy link
Contributor

Issue #, if available:

Description of changes:
Included in this PR is the update to support Windows just since that's what I'm developing on, but that's covered in this PR #276

Adding enum values for non-dynamic imports, and ensuring there's a single mapping location, so updates or changes downstream are safer to apply.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@aherschel aherschel requested a review from a team December 5, 2021 20:21
@codecov-commenter
Copy link

Codecov Report

Merging #277 (0e17335) into develop (eba0c24) will increase coverage by 0.22%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #277      +/-   ##
===========================================
+ Coverage    89.25%   89.48%   +0.22%     
===========================================
  Files           32       34       +2     
  Lines         1191     1217      +26     
  Branches       271      273       +2     
===========================================
+ Hits          1063     1089      +26     
  Misses         125      125              
  Partials         3        3              
Impacted Files Coverage Δ
.../codegen-ui-react/lib/imports/import-collection.ts 100.00% <100.00%> (ø)
...ges/codegen-ui-react/lib/imports/import-mapping.ts 100.00% <100.00%> (ø)
packages/codegen-ui-react/lib/imports/index.ts 100.00% <100.00%> (ø)
packages/codegen-ui-react/lib/index.ts 100.00% <100.00%> (ø)
...egen-ui-react/lib/react-component-render-helper.ts 75.75% <100.00%> (+0.10%) ⬆️
...s/codegen-ui-react/lib/react-component-renderer.ts 92.85% <100.00%> (+0.26%) ⬆️
...eact/lib/react-component-with-children-renderer.ts 94.00% <100.00%> (+0.12%) ⬆️
...-react/lib/react-index-studio-template-renderer.ts 43.33% <100.00%> (ø)
...gen-ui-react/lib/react-studio-template-renderer.ts 90.82% <100.00%> (ø)
...-react/lib/react-theme-studio-template-renderer.ts 87.23% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update eba0c24...0e17335. Read the comment docs.

@alharris-at alharris-at merged commit 9a9a78f into aws-amplify:develop Dec 6, 2021
dpilch pushed a commit that referenced this pull request Dec 6, 2021
…s import type. (#277)

* feat: add support for developing on windows

* chore: add import mapping for static import types

* fix: correct EscapeHatchProps import, and ensure it works with tsx integ tests
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.

None yet

4 participants