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

chore: extract test-app into own package #745

Merged
merged 1 commit into from
May 26, 2022

Conversation

SergeAstapov
Copy link
Contributor

@marcoow this is continuation of the work started in #740

This PR introduces new package in the monorepo test-app that holds the dummy test app and tests.

Removed from the packages/ember-cookies app related code and from the packages/test-app addon related code (however tried to do just minimum changes for dependencies and eslintrc file).

Some files marked as "new" nut they are actually copied from packages/ember-cookies to packages/test-app.

Copy link
Member

@marcoow marcoow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice 🙌

.github/workflows/ci.yml Show resolved Hide resolved
@SergeAstapov
Copy link
Contributor Author

@marcoow do you mind to take another look or delegate if there is anyone available?

Copy link
Member

@marcoow marcoow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@marcoow marcoow merged commit 6f0248b into mainmatter:master May 26, 2022
@SergeAstapov SergeAstapov deleted the extract-test-app branch May 26, 2022 11:50
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

2 participants