feat: use command replace io

[nonzzz]

Background

Description

• ReWrite WriteNpm func.

Others

[nonzzz]

@zaunist cc.

[codecov-commenter]

Codecov Report

Merging #23 (6db6dc3) into master (7523ace) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #23   +/-   ##
=======================================
  Coverage   75.00%   75.00%           
=======================================
  Files           2        2           
  Lines          20       20           
=======================================
  Hits           15       15           
  Misses          4        4           
  Partials        1        1           

Flag	Coverage Δ
unittests	75.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

[nonzzz]

You can view npm config docs. Can found npm config set registry uri. So i think we can use npm command replace past io operate. From my point, We use npm command can reduce unnecessary code of grm. And get more friendly err log from npm self. If user set a err uri.

[zaunist]

Why not use if err != nil ? Even though this style is silly, we should use a uniform style for our code. The use of if flag here also causes the call to writeNpm to return 0 and return 1 in a different style.

[nonzzz]

You are right. May we can discuss how to unify the norms.Because in project. we have process state code. In my mind. Just like process state code we can provide a enum,
It look like

eg:

const (

NORMAL = 1 << iota
ABORT
)

And for other err. May if err !=nil. My mind don't have an answer for now :(

[zaunist]

Make sense to me.

[nonzzz]

I push new code. This commit I just kept the err return at write_ini.go file.

[nonzzz]

@zaunist cc

[zaunist]

LGTM