Skip to content

Conversation

@yaricom
Copy link
Owner

@yaricom yaricom commented May 29, 2022

No description provided.

@codecov
Copy link

codecov bot commented May 29, 2022

Codecov Report

Merging #4 (c93260a) into master (f6d6d12) will increase coverage by 0.31%.
The diff coverage is 44.44%.

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
+ Coverage   39.31%   39.62%   +0.31%     
==========================================
  Files           9        9              
  Lines         702      699       -3     
==========================================
+ Hits          276      277       +1     
+ Misses        402      400       -2     
+ Partials       24       22       -2     
Impacted Files Coverage Δ
examples/maze/maze_ns.go 0.00% <0.00%> (ø)
examples/maze/maze_obj.go 0.00% <0.00%> (ø)
neatns/novelty_archive.go 80.00% <ø> (ø)
neatns/novelty_archive_io.go 71.42% <63.63%> (+21.42%) ⬆️
neatns/novelty_item.go 80.76% <100.00%> (-0.72%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f6d6d12...c93260a. Read the comment docs.


// decode and check
var novelItems []*NoveltyItem
err = json.Unmarshal(buf.Bytes(), &novelItems)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SA4006: this value of err is never used

(at-me in a reply with help or ignore)


Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]


// decode and check
var novelItems []*NoveltyItem
err = json.Unmarshal(buf.Bytes(), &novelItems)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ineffassign: ineffectual assignment to err

(at-me in a reply with help or ignore)


Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]


// decode and check
var novelItems []*NoveltyItem
err = json.Unmarshal(buf.Bytes(), &novelItems)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SA4006: this value of err is never used

(at-me in a reply with help or ignore)


Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]


// decode and check
var novelItems []*NoveltyItem
err = json.Unmarshal(buf.Bytes(), &novelItems)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ineffassign: ineffectual assignment to err

(at-me in a reply with help or ignore)


Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]

@yaricom yaricom merged commit 4188d5c into master May 29, 2022
@yaricom yaricom deleted the write-novelty-to-json branch May 29, 2022 17:47
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