Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Testing new macos lane #3953

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

Merged
merged 6 commits into from
Nov 20, 2020
Merged

Conversation

mdh1418
Copy link
Member

@mdh1418 mdh1418 commented Nov 13, 2020

Summary

With the import of dotnet/runtime samples that demonstrate how to build iOS, Android, and wasm apps on top of the mono runtime, a check to ensure that these have built properly is needed. The mono samples need to pull down a runtime pack to properly build the application, so dotnet publish will be used instead of dotnet build.

The applications will not be ran for these checks. Ensuring that the applications can be built is the primary goal.

This PR looks to add another GitHub Actions workflow that targets these mono-samples.
When a file that influences the publish process is changed, the original build shall not be ran for the project, instead, the new one will be ran.

As seen in this PR, the original build check does not run for this PR, only Publish Mono Samples is ran.

@mdh1418 mdh1418 force-pushed the mdhwang/add_macos_ci_lane branch from 34d6ea2 to 04612e0 Compare November 13, 2020 20:54
@mdh1418 mdh1418 added the WIP Work in progress label Nov 13, 2020
@mdh1418 mdh1418 marked this pull request as ready for review November 13, 2020 20:55
@mdh1418 mdh1418 requested a review from a team as a code owner November 13, 2020 20:55
@mdh1418 mdh1418 force-pushed the mdhwang/add_macos_ci_lane branch from 04612e0 to 46c775a Compare November 13, 2020 20:59
@mdh1418 mdh1418 marked this pull request as draft November 13, 2020 21:00
@mdh1418 mdh1418 removed the request for review from a team November 13, 2020 21:00
@mdh1418 mdh1418 force-pushed the mdhwang/add_macos_ci_lane branch 5 times, most recently from b91aabd to 9d8f659 Compare November 16, 2020 20:52
@mdh1418 mdh1418 marked this pull request as ready for review November 16, 2020 20:55
@mdh1418
Copy link
Member Author

mdh1418 commented Nov 16, 2020

Under the scenario where both a mono sample and non-mono sample are modified in the same pull request, is there a way to run the original build validation without the mono sample changes?
How can we bypass building mono samples on the windows vm?
I don't think

    paths-ignore:
    - 'core/mono-samples/**'

achieves this.
Perhaps core/mono-samples/** can be omitted in Locate projects for PR somehow?

@akoeplinger
Copy link
Member

Perhaps core/mono-samples/** can be omitted in Locate projects for PR somehow?

I think you'll need to tweak the Get-MSBuildResults.ps1 script to exclude certain paths.

@mdh1418 mdh1418 force-pushed the mdhwang/add_macos_ci_lane branch from 1f8737e to c97b717 Compare November 17, 2020 19:20
@mdh1418 mdh1418 force-pushed the mdhwang/add_macos_ci_lane branch from c97b717 to 58ae03a Compare November 17, 2020 19:21
@mdh1418 mdh1418 removed the WIP Work in progress label Nov 17, 2020
@mdh1418 mdh1418 force-pushed the mdhwang/add_macos_ci_lane branch from 7a44a71 to e72c8fc Compare November 18, 2020 17:34
@mdh1418 mdh1418 force-pushed the mdhwang/add_macos_ci_lane branch from e72c8fc to 7ab52c6 Compare November 18, 2020 18:02
@mdh1418 mdh1418 merged commit 7ae0606 into dotnet:master Nov 20, 2020
@mdh1418 mdh1418 deleted the mdhwang/add_macos_ci_lane branch November 20, 2020 15:21
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