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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Intuita 1-Click Run #77

Merged
merged 6 commits into from
Sep 15, 2023
Merged

Added Intuita 1-Click Run #77

merged 6 commits into from
Sep 15, 2023

Conversation

mohab-sameh
Copy link
Contributor

Added support for running Ratchet using the Intuita VS Code extension. Now you can 1-click run Ratchet using Intuita VSCE on your projects.

This addition allows you to use Intuita's features with Ratchet, such as:

  • Integration with Intuita's platform.
  • Dry-run Ratchet (and other codemods).
  • Review and edit changes made by the codemod.
  • Report issues with codemod accuracy, and get help from a community of codemod experts.
  • Target paths/files during codemod runs.

馃挕 To learn more about the features you can use while running Ratchet with Intuita's VSCE, read the docs here.

@mskelton
Copy link
Owner

mskelton commented Sep 9, 2023

@mohab-sameh This looks great. My only concern is around the language of it. I'm a little concerned that it ties a direct connection between Ratchet and Intuita which could seem to indicate that I'm employed by y'all or something similar. Not that it's necessarily bad or would harm either me or you guys, but it could be a little confusing for users.

Totally fine with it being the first thing before the CLI command, I love what you all are doing and thing it's a great workflow for codemods.

@mohab-sameh
Copy link
Contributor Author

@mohab-sameh This looks great. My only concern is around the language of it. I'm a little concerned that it ties a direct connection between Ratchet and Intuita which could seem to indicate that I'm employed by y'all or something similar. Not that it's necessarily bad or would harm either me or you guys, but it could be a little confusing for users.

Totally fine with it being the first thing before the CLI command, I love what you all are doing and thing it's a great workflow for codemods.

Totally understandable. It would be great if we could let users know that the codemod is running on a different platform (Intuita) so that they are aware they will be redirected elsewhere. But definitely, we can tone down the branding if you like or put a non-affiliation note. I can change this or feel free to edit it to something you consider acceptable before merging. Thanks a lot for bringing it up @mskelton

@mskelton mskelton merged commit 8b832d0 into mskelton:main Sep 15, 2023
2 of 3 checks passed
@mskelton
Copy link
Owner

@mohab-sameh Sorry for the delay, busy week for me, but I made a few adjustments to the text and merged this in!

@mohab-sameh
Copy link
Contributor Author

@mskelton no worries. awesome!

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

Successfully merging this pull request may close these issues.

None yet

2 participants