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

ACT: introduce Share in Playground action #6563

Merged
merged 5 commits into from Jan 21, 2021

Conversation

Undin
Copy link
Member

@Undin Undin commented Dec 27, 2020

These changes:

  • Introduce Share in Playground action to share your code in https://play.rust-lang.org/. The action correctly handles selected text in editor (i.e. share only selected text), current edition and toolchain channel.
  • Add Rust action group to have a single item in context menus for util Rust actions. Reformat File with Rustfmt, Reformat Cargo Project with Rustfmt, Rust REPL, Share in Playground are placed into this group as well as Show expanded Macro actions. Also, the group is added to Tools menu.
  • Provide MockServerFixture to help setting up mock web server in test to check network interactions
Screen.Recording.2020-12-28.at.02.39.43.mov

changelog: Introduce Share in Playground action to share your code in https://play.rust-lang.org/. You can invoke it via Tools | Rust | Share in Playground or via context menu

@Undin Undin added the feature label Dec 27, 2020
@Undin Undin added this to In Progress in To test via automation Dec 27, 2020
@Undin Undin requested a review from vlad20012 December 27, 2020 23:53
@Undin Undin force-pushed the undin/share-in-playground-action branch from 6fbeb53 to 9289c21 Compare December 28, 2020 23:47
@artemmukhin
Copy link
Member

Probably it's worth showing a confirmation modal window before sending the code to the server. I'm afraid someone might accidentally invoke this action, and some potentially proprietary code will immediately go to the server. In the modal window, we can also provide some information like "entire file filename.rs / selected code fragment will be sent to the server"

@Undin Undin added the to be documented Pull requests that should be documented label Jan 15, 2021
@Undin
Copy link
Member Author

Undin commented Jan 15, 2021

@ortem done

@@ -286,6 +286,7 @@ project(":") {
exclude(module = "kotlin-stdlib")
}
testImplementation(project(":common", "testOutput"))
testImplementation("com.squareup.okhttp3:mockwebserver:4.9.0")
Copy link
Member

Choose a reason for hiding this comment

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

Wow, it's cool

Copy link
Member

@artemmukhin artemmukhin left a comment

Choose a reason for hiding this comment

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

LGTM except for several minor comments

@Undin Undin force-pushed the undin/share-in-playground-action branch from 636308b to e0f676a Compare January 21, 2021 22:16
@Undin
Copy link
Member Author

Undin commented Jan 21, 2021

bors r=ortem

@bors
Copy link
Contributor

bors bot commented Jan 21, 2021

Build succeeded:

@bors bors bot merged commit 25ad5e8 into master Jan 21, 2021
To test automation moved this from In Progress to Test Jan 21, 2021
@bors bors bot deleted the undin/share-in-playground-action branch January 21, 2021 23:57
@github-actions github-actions bot added this to the v140 milestone Jan 21, 2021
@lancelote lancelote moved this from Test to Done in To test Mar 15, 2021
@KseniaStar KseniaStar removed the to be documented Pull requests that should be documented label Oct 29, 2022
@KseniaStar
Copy link
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
To test
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants