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

Don't implicitly convert . & .syntuxignore #10

Open
seidtgeist opened this issue Nov 22, 2011 · 6 comments
Open

Don't implicitly convert . & .syntuxignore #10

seidtgeist opened this issue Nov 22, 2011 · 6 comments

Comments

@seidtgeist
Copy link
Owner

@felixge mentioned respecting contents of .gitignore, a potential .syntuxignore and also ignoring folders like node_modules

@klipstein recommended disabling the default behaviour of recursively converting the current dir

Edit: Herped the derp

@seidtgeist
Copy link
Owner Author

Ok, here's my personal reasoning:

Because syntux is a program which modifies files we might want to disable the implicit path = current dir behavior. Let's compare this with *nix: ls for example clearly doesn't need a path. rm on the other hand, well, what can I say, that's a stupid example. sed and awk need arguments. chmod and chown need arguments. If we changed that, to achieve the same result one could simply write syntux . instead of syntux. I think that's bearable.

Additionally I think a .syntuxignore file would make sense. Maintaining an internal list of common folders might be a never ending task. Maybe rather provide/create an example .syntuxignore in a syntux --init command which would create syntax.json, too?

@felixge
Copy link
Collaborator

felixge commented Nov 23, 2011

I'm ok with not assuming the CWD as the path. Lets change that.

@seidtgeist
Copy link
Owner Author

Might as well make this an stdin to stdout mode.

@felixge
Copy link
Collaborator

felixge commented Nov 25, 2011

Either that, or a REPL.

@seidtgeist
Copy link
Owner Author

+1

Can we detect how syntux_cli is being used from the outside? Thinking "pipe vs repl mode".

@felixge
Copy link
Collaborator

felixge commented Nov 28, 2011

Can we detect how syntux_cli is being used from the outside? Thinking "pipe vs repl mode".

I think so. If not we'll have a switch for the repl. Piping should be given more importance.

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

2 participants