Skip to content

Update appReexports#738

Merged
mkszepp merged 1 commit intocibernox:masterfrom
mkszepp:update-app-reexport
Jan 11, 2024
Merged

Update appReexports#738
mkszepp merged 1 commit intocibernox:masterfrom
mkszepp:update-app-reexport

Conversation

@mkszepp
Copy link
Copy Markdown
Collaborator

@mkszepp mkszepp commented Jan 11, 2024 •
edited
Loading

There is not necessary to add appReexport for test-support

This brings warnings in embroider build process like:

WARNING in ../../../../ember-basic-dropdown/dist/_app_/test-support/helpers.js 1:0-68
export 'default' (reexported as 'default') was not found in 'ember-basic-dropdown/test-support/helpers' (possible exports: clickTrigger, tapTrigger)
 @ ./assets/test-app.js 16:13-56

@mkszepp mkszepp added the bug label Jan 11, 2024
@mkszepp mkszepp merged commit c0ada87 into cibernox:master Jan 11, 2024
@mkszepp mkszepp deleted the update-app-reexport branch January 11, 2024 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

bug

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@mkszepp