Skip to content
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

Run build & test steps on windows & macos if the ci/test label is set #227

Merged
merged 2 commits into from
Jan 29, 2024

Conversation

julienp
Copy link
Contributor

@julienp julienp commented Jan 29, 2024

Currently we only run a certain integration tests on windows & Mac.

Allow running the sdk and automation tests on windows & Mac by setting the ci/test label.

@julienp julienp force-pushed the julienp/run-build-test-label branch 9 times, most recently from 4831c7f to 61393ef Compare January 29, 2024 10:15
@julienp julienp force-pushed the julienp/run-build-test-label branch 5 times, most recently from c4adea2 to 60ebcb5 Compare January 29, 2024 10:34
@julienp julienp force-pushed the julienp/run-build-test-label branch 3 times, most recently from a963451 to 05cc51a Compare January 29, 2024 10:40
@julienp julienp force-pushed the julienp/run-build-test-label branch 4 times, most recently from dc96cc5 to ff6e0f7 Compare January 29, 2024 11:22
@julienp julienp added the impact/no-changelog-required This issue doesn't require a CHANGELOG update label Jan 29, 2024
@julienp julienp marked this pull request as ready for review January 29, 2024 13:02
Copy link
Contributor

@tgummerer tgummerer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@julienp julienp force-pushed the julienp/run-build-test-label branch from ff6e0f7 to bb8efee Compare January 29, 2024 13:47
We don't want it to complain about line endings on windows.
@julienp julienp force-pushed the julienp/run-build-test-label branch from bb8efee to fa04622 Compare January 29, 2024 13:48
@julienp julienp enabled auto-merge January 29, 2024 14:24
@julienp julienp added this pull request to the merge queue Jan 29, 2024
Merged via the queue into main with commit 557d710 Jan 29, 2024
19 checks passed
@julienp julienp deleted the julienp/run-build-test-label branch January 29, 2024 15:13
jkerken pushed a commit to algompluecker/pulumi-dotnet that referenced this pull request Jul 24, 2024
…pulumi#227)

* Run build & test steps on windows & macos if the ci/test label is set

* Only run formatting check on linux

We don't want it to complain about line endings on windows.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/test impact/no-changelog-required This issue doesn't require a CHANGELOG update
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants