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

Added json output function #389

Merged
merged 3 commits into from
May 2, 2024
Merged

Added json output function #389

merged 3 commits into from
May 2, 2024

Conversation

fksms
Copy link
Contributor

@fksms fksms commented Apr 25, 2024

I added it because I needed the ability to output it as json.

dust -j outputs json to the current directory.

For your reference.

@bootandy
Copy link
Owner

Ok, lets fix the format and go

@fksms
Copy link
Contributor Author

fksms commented May 2, 2024

Thank you for confirmation.

What specifically should I fix?

@bootandy
Copy link
Owner

bootandy commented May 2, 2024

Run cargo fmt locally - you can see the build failed above ^ , then push again.

@bootandy
Copy link
Owner

bootandy commented May 2, 2024

and rebase

@fksms
Copy link
Contributor Author

fksms commented May 2, 2024

Very helpful.

I've Done!

@bootandy bootandy merged commit 2c34c38 into bootandy:master May 2, 2024
17 checks passed
@bootandy
Copy link
Owner

bootandy commented May 2, 2024

I'm going to have to tweak this a bit, it's not doing what I expected.

@bootandy
Copy link
Owner

bootandy commented May 2, 2024

#396
Here, we print to stdout instead - I don't want my tool writing files to the disk. - if you want the old behaviour you can do
dust > node.json

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.

2 participants