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

nimpretty status? #7420

Closed
timotheecour opened this Issue Mar 27, 2018 · 6 comments

Comments

Projects
None yet
6 participants
@timotheecour
Copy link
Collaborator

timotheecour commented Mar 27, 2018

I'm looking for a tool like D's dfmt --inplace or gofmt to format code.
I found nimpretty but couldn't find much information on it;
it's not installed via brew install nim (which installs nim and nimble and nimgrep) which makes me think it's not ready yet.

just found these links:

so is that feature planned, what works/doesn't?

also, shouldn't dedicated github issue project be used for project specific things such as nimpretty? (eg there's no existing label nimpretty)

@dom96

This comment has been minimized.

Copy link
Member

dom96 commented Mar 27, 2018

also, shouldn't dedicated github issue project be used for project specific things such as nimpretty? (eg there's no existing label nimpretty)

Yep, and a dedicated repo for the project should be made too. @Araq prefers to keep things in the Nim repo though :/

@Araq

This comment has been minimized.

Copy link
Member

Araq commented Apr 23, 2018

so is that feature planned, what works/doesn't?

Planned and the biggest blocking issue is still comment handling. Once that is solved, I will run it on every .nim file I can get my hands on and try to get it into production quality.

@dom96

This comment has been minimized.

Copy link
Member

dom96 commented Apr 23, 2018

Will you also move it into a Nimble repo to make contribution easier?

@GULPF

This comment has been minimized.

Copy link
Member

GULPF commented Apr 23, 2018

@dom96 nimpretty is baked into the compiler, so I doubt that's possible. All the work is done by compiler/parser and compiler/renderer.

@data-man

This comment has been minimized.

Copy link
Collaborator

data-man commented Apr 23, 2018

@GULPF
c2nim also depends from the compiler's modules, but can be installed using Nimble.
So I don't think that this is a problem.

@Araq

This comment has been minimized.

Copy link
Member

Araq commented Sep 2, 2018

Question has been answered.

@Araq Araq closed this Sep 2, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment