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

Check if style file with project name exists #3642

Merged
merged 2 commits into from Mar 26, 2015
Merged

Check if style file with project name exists #3642

merged 2 commits into from Mar 26, 2015

Conversation

btecu
Copy link
Contributor

@btecu btecu commented Mar 25, 2015

Currently if a style sheet file named the same as the project exists will cause an exception when building. The error is not very obvious. This check prevents the build and displays an error.

Currently if a style sheet file named the same as the project exists will cause an exception when building. The error is not very obvious. This check prevents the build and displays an error.
@rwjblue
Copy link
Member

rwjblue commented Mar 25, 2015

Seems good to provide a nice warning here. It will only be checked on initial build (which seems OK to me).

@stefanpenner - r?

@stefanpenner
Copy link
Contributor

@rwjblue should this be a silent error?

@rwjblue
Copy link
Member

rwjblue commented Mar 25, 2015

Yes, definitely (good catch!). @btecu - Can you update to use SilentError?

@btecu
Copy link
Contributor Author

btecu commented Mar 25, 2015

Done.

@stefanpenner
Copy link
Contributor

re-kicking app-veyor

@stefanpenner stefanpenner added this to the v0.2.2 milestone Mar 25, 2015
rwjblue added a commit that referenced this pull request Mar 26, 2015
Check if style file with project name exists
@rwjblue rwjblue merged commit 38ad045 into ember-cli:master Mar 26, 2015
@btecu btecu deleted the css branch March 29, 2015 18:06
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

3 participants