-
Notifications
You must be signed in to change notification settings - Fork 10.4k
See what happens with blazor build in helix #18962
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
Conversation
@pranavkm I reenabled these tests on helix but it looks like they will need some rejiggering to work, as they are looking for the files based on build time locations, can you see if you can make any tactical changes to make these tests work on helix (its valid to copy the test assets into the public directory, and then have the test look for them using a relative path to the runtime directory). only 13 tests are failing:
|
ea2fcd3
to
20c3e46
Compare
@pranavkm I fixed some the path issues, now it looks like there are some harder ones where the test projects are referencing ReferenceBlazorBuildFromSource.props which has paths hardcoded, so its looking for a non existent versions.props. Any ideas how to hijack this logic to make it look for a hardcoded path since we can include the necessary files in the helix payload so relative paths would work always.
|
We could make this line conditional: - <Import Project="$(RepoRoot)eng\Versions.props" />
+ <Import Project="$(RepoRoot)eng\Versions.props" Condition="Exists($(RepoRoot)eng\Versions.props))" />
+ <Import Project="$(SomeMagicDirThatExistsInHelix)Versions.props" Condition="Exists($(SomeMagicDirThatExistsInHelix)Versions.props))" /> And copy the |
@pranavkm tried your suggestion and it looks like it might cause issues with restore?
|
cf1fd3d
to
975ea59
Compare
975ea59
to
4aa4bfb
Compare
1fd29d6
to
facd961
Compare
e7e425d
to
7d916aa
Compare
Rebasing to bring in the latest changes now that the blazor-wasm branch is merged. Let's see what chaos ensues.... |
7560e56
to
c4f8349
Compare
10c3977
to
6ea822f
Compare
6ea822f
to
2bd2721
Compare
49c1820
to
1d140df
Compare
Closing this PR. Our build tests have change a bit now that we have a Blazor WASM SDK. Will open a new PR with changes. |
No description provided.