Skip to content
This repository has been archived by the owner on Apr 25, 2023. It is now read-only.

Check for error in Fprintf #51

Closed
wants to merge 1 commit into from
Closed

Check for error in Fprintf #51

wants to merge 1 commit into from

Conversation

Sohalt
Copy link
Contributor

@Sohalt Sohalt commented Apr 13, 2020

No description provided.

@Sohalt
Copy link
Contributor Author

Sohalt commented Apr 13, 2020

The Review code stage should go through now. Can you restart the pipeline please?

@spetzreborn
Copy link
Contributor

I accidentally included this no error check bug in cea0434
After more reading, both docs and this project I think it is more canon to use json.NewEncoder().Encode() instead.

I've included this fix, as well as error checking in the PR #49
But there are still buildfailes that I don't know ho to fix.

@Sohalt Sohalt closed this Apr 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Sohalt @spetzreborn