Add RWX build provider#4836
Open
robinaugh wants to merge 1 commit into
Open
Conversation
Author
|
@microsoft-github-policy-service agree company="RWX" |
devlead
force-pushed
the
jason/add-rwx-build-provider
branch
from
59cab62 to
7742642
Compare
devlead
reviewed
May 20, 2026
Member
devlead
left a comment
devlead
left a comment
There was a problem hiding this comment.
In general LGTM 👍 , just a few questions.
robinaugh
force-pushed
the
jason/add-rwx-build-provider
branch
2 times, most recently
from
29a66b3 to
8521060
Compare
Adds Cake.Common.Build.Rwx alongside the 14 existing build providers,
exposing IsRunningOnRwx and BuildSystem.Rwx.Environment.{Run,Task,Actor}
populated from the documented RWX env var contract
(https://www.rwx.com/docs/environment-variables). Detection is via the
RWX env var; RWX does not currently expose a PR identifier so this
provider does not contribute to BuildSystem.IsPullRequest.
devlead
force-pushed
the
jason/add-rwx-build-provider
branch
from
8521060 to
84d919b
Compare
devlead
requested changes
May 20, 2026
|
|
||
| Task("Cake.Common.Build.RwxProvider.Commands") | ||
| .Does(() => { | ||
| BuildSystem.Rwx.Commands.SetValue("cake_integration_test", "ok"); |
Member
There was a problem hiding this comment.
UploadArtifact works great
But how do we validate SetValue? Could we have an extra task in .rwx/cake.yml that echoes / asserts cake_integration_test ?
| /// are written verbatim. | ||
| /// </summary> | ||
| /// <param name="key">The value key. May not contain path separators or <c>..</c>.</param> | ||
| /// <param name="value">The value contents.</param> |
Member
There was a problem hiding this comment.
Looking at docs, we could have similar methods for env vars
https://www.rwx.com/docs/use#environment-variables
tasks:
- key: set-env-one
run: echo hello >> $RWX_ENV/ONE
- key: set-env-two
run: echo world >> $RWX_ENV/TWO
- key: print-env
use: [set-env-one, set-env-two]
run: |
echo $ONE $TWO
env | grep -E 'ONE|TWO'
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Following https://github.com/orgs/cake-build/discussions/4796, this PR adds RWX as a build provider.
I have already been in contact with @devlead via email and we've set up a Cake organization on RWX. A sample run can be seen here: https://cloud.rwx.com/mint/cake/runs/a8eb7be0436248508544a2c0fda9df08
main/develop/hotfix*branches) will be publicly accessible and linked from the badge in the README.