-
Notifications
You must be signed in to change notification settings - Fork 10.5k
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.