eXist-db / xst

Command line tool to interact with exist-db instances
https://npmjs.com/@existdb/xst
MIT License
18 stars 5 forks source link

Preparations for initial release #22

Closed line-o closed 1 year ago

line-o commented 2 years ago

This will set the package name to @existdb/xst. Merging this PR will trigger the release to npm which currently fails due to a naming conflict.

closes #6, closes #9, fixes #11, closes #18, closes #19, closes #20, closes #23, closes #24, closes #29, closes #33, closes #34, closes #35, closes #36, closes #38, closes #42, closes #43, closes #44, closes #45

should be closed first

line-o commented 1 year ago

I would like to invite interested parties for a final review before the initial release. Is the readme clear and concise? @joewiz @duncdrum @cwittern

duncdrum commented 1 year ago

@line-o yup looks good

joewiz commented 1 year ago

p.s. Any idea why xst returns the following results?

% xst run '(1 to 10) ! string-join(., "
")' 
1,2,3,4,5,6,7,8,9,10

I'd have expected the numbers to be separated by newlines. I assume this is by design but can't explain why/how this is, so I thought I'd raise it here in case it's unexpected to you too.

joewiz commented 1 year ago

Oops, my bad. Sorry for the noise.

xst run '(1 to 10) => string-join("
")'
1
2
3
4
5
6
7
8
9
10
line-o commented 1 year ago

Your first query should work too.

line-o commented 1 year ago

@joewiz Ah, no I see why the first one could not work. :)