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

STDIN support? #53

Closed
alok opened this issue Mar 13, 2018 · 5 comments
Closed

STDIN support? #53

alok opened this issue Mar 13, 2018 · 5 comments

Comments

@alok
Copy link

alok commented Mar 13, 2018

Just saw this, and I'd love to incorporate it into plugins like neoformat. It'd be nice if pyment could take in code via STDIN and output the newly formatted code (not a patch but just the updated code) via STDOUT. Essentially, the -w flag without actually overwriting a file.

@dadadel
Copy link
Owner

dadadel commented Apr 26, 2018

Hi @alok,
This is a good idea indeed. Feel free to contribute if you can, else I'll try to find some time to do so.
Thanks for your issue.

@bryder
Copy link
Contributor

bryder commented Jun 16, 2018

I have submitted #59 to implement this.

dadadel added a commit that referenced this issue Jun 17, 2018
@dadadel
Copy link
Owner

dadadel commented Jun 17, 2018

Thanks @alok for the suggestion and thanks @bryder for the implementation!

I close as it works well!

@dadadel dadadel closed this as completed Jun 17, 2018
@dadadel dadadel reopened this Jun 17, 2018
@dadadel
Copy link
Owner

dadadel commented Jun 17, 2018

As pointed out by @alok it sounds better if the default output is actually the new version of the input instead of the patch.

@dadadel
Copy link
Owner

dadadel commented Mar 8, 2021

I think this issue can be closed as I reopened it for a bad reason!
If someone think the default behavior of Pyment should be changed so an other issue would be relevant.

@dadadel dadadel closed this as completed Mar 8, 2021
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

No branches or pull requests

3 participants