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

AST output format; new configuration options & fix some bugs #132

Merged
merged 6 commits into from
May 29, 2023

Conversation

stdword
Copy link
Contributor

@stdword stdword commented Mar 28, 2023

  • fix: stdout cannot be used as output
  • fix: read from stdin doesn't work
  • fix: html output format cannot be used
  • fix: arg append is not a boolean type
  • feat: added -f, -format param → output format can be different than extension
  • feat: added ast output format
  • feat: added -with-pos option to include positions meta info

- There were no way to use `stdout` as output
- Added `-f`, `-format` param → output format can be `html`
@CLAassistant
Copy link

CLAassistant commented Mar 28, 2023

CLA assistant check
All committers have signed the CLA.

@stdword stdword changed the title fix(cli): bug with necessarity argv.o , argv.i AST output format & fix bug with required argv.o, argv.i Mar 28, 2023
@stdword stdword changed the title AST output format & fix bug with required argv.o, argv.i AST output format & fix bug with required argv.o, argv.i Mar 28, 2023
@stdword stdword changed the title AST output format & fix bug with required argv.o, argv.i AST output format; new configuration options & fix some bugs Mar 28, 2023
@stdword
Copy link
Contributor Author

stdword commented Apr 12, 2023

@tiensonqin look at this PR, please 🙏

@Bad3r
Copy link

Bad3r commented Apr 13, 2023

@RCmerci Please take a look at this PR when you get a chance :)

@garyo
Copy link

garyo commented Apr 25, 2023

Per #125 I would love to see this merged, if it can output org-mode! (I tried to build it but I'm not familiar enough with the ocaml toolchain.)

@stdword
Copy link
Contributor Author

stdword commented Apr 25, 2023

I've recheck it and unfortunately it cannot do the conversion md <-> org. So I've fixed only the js script, not a referenced bug =(

So the js script only can make a conversion to AST

@tiensonqin tiensonqin merged commit ac0dceb into logseq:master May 29, 2023
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.

5 participants