Fix merge error if there are two files with the same name#15
Fix merge error if there are two files with the same name#15tschoartschi wants to merge 2 commits intotomasbasham:masterfrom
Conversation
This makes it possible that there can be files with the same name in addon and tests/dummy/app folder
|
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? |
|
I forgot to mention that you need to specify the following options: scssLintOptions: {
includePaths: [
'addon'
]
}Otherwise the files in |
|
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. |
|
@tomasbasham cool. I'm happy that I could help you reproduce the bug. Let me know if I can help any further |
|
Any news on this issue / merge request? |
|
We have a very similar issue. This pull request seems to fix our problems. Is it good to merge? |
|
The tests are not passing so I'll need to investigate why first. |
|
@tomasbasham thanks for the quick response. I can use @tschoartschi branch in the meantime |
b489569 to
d67d097
Compare
I develop an addon which I test with the
tests/dummy/app. If there is a file with the same name inaddon/stylesandtests/dummy/app/stylesthe 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: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 😃