Skip to content
This repository was archived by the owner on Dec 16, 2024. It is now read-only.

Comments

Fix Windows minion targeting#785

Merged
bors-servo merged 1 commit intoservo:masterfrom
aneeshusa:fix-windows-minion-targeting
Jan 28, 2018
Merged

Fix Windows minion targeting#785
bors-servo merged 1 commit intoservo:masterfrom
aneeshusa:fix-windows-minion-targeting

Conversation

@aneeshusa
Copy link
Contributor

@aneeshusa aneeshusa commented Jan 28, 2018

Use the correct type of targeting so the Windows minions
actually have states to run during highstates.

Fixes #766.


This change is Reviewable

Use the correct type of targeting so the Windows minions
actually have states to run during highstates.
@aneeshusa
Copy link
Contributor Author

I want to get this in to make it easier to apply and test sccache changes.
r? @larsbergstrom or @jdm

@jdm
Copy link
Member

jdm commented Jan 28, 2018

@bors-servo r+
Geeze. I feel silly for missing that.

@bors-servo
Copy link
Contributor

📌 Commit 30ba649 has been approved by jdm

@bors-servo
Copy link
Contributor

⌛ Testing commit 30ba649 with merge ee0a6fb...

bors-servo pushed a commit that referenced this pull request Jan 28, 2018
Fix Windows minion targeting

Use the correct type of targeting so the Windows minions
actually have states to run during highstates.

Fixes #766.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/785)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

☀️ Test successful - status-travis
Approved by: jdm
Pushing ee0a6fb to master...

@bors-servo bors-servo merged commit 30ba649 into servo:master Jan 28, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants