Skip to content

Conversation

tschoartschi
Copy link

@tschoartschi tschoartschi commented Jul 27, 2017

I develop an addon which I test with the tests/dummy/app. If there is a file with the same name in addon/styles and tests/dummy/app/styles the build crashes. To reproduce the crash just create a file with the name conflict.scss in both directories. The you will see the following exception:

Error: Merge error: file styles/conflict.sass-lint-test.js exists in /Users/user/Projects/webclient/tmp/broccoli_merge_trees-input_base_path-pfqHcQSE.tmp/0 and /Users/user/Projects/webclient/tmp/broccoli_merge_trees-input_base_path-pfqHcQSE.tmp/1
Pass option { overwrite: true } to mergeTrees in order to have the latter file win.

This pull request tries to fix this issue. I'm not an expert with the broccoli build tool but this fix resolves my issue. I'm not sure if this breaks anything else especially not because I was not able to get the test suite running. Please have a look and let me know what you think 😃

This makes it possible that there can be files with the same name
in addon and tests/dummy/app folder
@tomasbasham
Copy link
Owner

What versions of Node and Ember are you using? I cannot seem to reproduce this error. I have created a small test addon. Is this at least similar to your addon setup?

@tschoartschi
Copy link
Author

tschoartschi commented Jul 28, 2017

I forgot to mention that you need to specify the following options:

scssLintOptions: {
  includePaths: [
    'addon'
  ]
}

Otherwise the files in addon/styles are not linted. My current setup is

ember-cli: 2.13.2
node: 8.1.3
npm: 5.0.3
os: darwin x64
ember-cli-scss-lint: tried master and 2.2.0
ember-cli-sass: 6.2.0

@tomasbasham
Copy link
Owner

OK, got it. There is actually another issue related to the test files #14 that I need to investigate further. Although quite separate from this issue I will have to rethink how these files are generated.

@tschoartschi
Copy link
Author

@tomasbasham cool. I'm happy that I could help you reproduce the bug. Let me know if I can help any further

@tschoartschi
Copy link
Author

Any news on this issue / merge request?

@roomle-build
Copy link

We have a very similar issue. This pull request seems to fix our problems. Is it good to merge?

@tomasbasham
Copy link
Owner

The tests are not passing so I'll need to investigate why first.

@roomle-build
Copy link

@tomasbasham thanks for the quick response. I can use @tschoartschi branch in the meantime

@tomasbasham tomasbasham force-pushed the master branch 5 times, most recently from b489569 to d67d097 Compare January 20, 2019 17:25
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.

3 participants