-
Notifications
You must be signed in to change notification settings - Fork 390
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
Allow to run arbitrary commands in containers using cross-util run ... #1280
Allow to run arbitrary commands in containers using cross-util run ... #1280
Conversation
1fb1f17
to
f09361c
Compare
9623db9
to
278c3ea
Compare
278c3ea
to
be92c3f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
awesome, some pointers
7e46941
to
00c39a6
Compare
The changelog check is failing, I added a line in the changelog. Any suggestions on how to fix it? |
See https://github.com/cross-rs/cross/blob/main/.changes/README.md for how to make a changelog entry :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm! fix the changeog and this should be good to go!
00c39a6
to
1aab4cf
Compare
Thanks, done 😄 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
awesome! thank you :)
bors r+
1280: Allow to run arbitrary commands in containers using cross-util run ... r=Emilgardis a=lumasepa Implementation for #1266 Co-authored-by: Sergio Medina <[email protected]>
I keep forgetting some targets are broken... bors cancel bors r+ |
Canceled. |
Build succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page. |
Implementation for #1266