Skip to content

fix: bump linter 1.48.0 and remplace "io/ioutil". #2447

Merged
Monitob merged 7 commits intoscaleway:masterfrom
Monitob:fix-linter
Aug 11, 2022
Merged

fix: bump linter 1.48.0 and remplace "io/ioutil". #2447
Monitob merged 7 commits intoscaleway:masterfrom
Monitob:fix-linter

Conversation

@Monitob
Copy link
Copy Markdown
Contributor

@Monitob Monitob commented Aug 10, 2022

Error: "io/ioutil" has been deprecated since Go 1.16: As of Go 1.16, the same functionality is now provided by package io or package os, and those implementations should be preferred in new code. See the specific function documentation for details.

Community Note

  • Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request.
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for pull request followers and do not help prioritize the request

Relates OR Closes #0000

Release note for CHANGELOG:


@Monitob Monitob changed the title fix: lister and remplace "io/ioutil". This pakage has been deprecated… fix: linter and remplace "io/ioutil". This pakage has been deprecated… Aug 10, 2022
@Monitob Monitob changed the title fix: linter and remplace "io/ioutil". This pakage has been deprecated… fix: bump linter 1.48.0 and remplace "io/ioutil". Aug 10, 2022
Comment thread internal/docgen/docgen.go Outdated
Comment thread internal/namespaces/baremetal/v1/baremetal_cli_test.go Outdated
Comment thread internal/namespaces/config/commands.go Outdated
Comment thread internal/namespaces/instance/v1/instance_cli_test.go Outdated
Vinzius
Vinzius previously approved these changes Aug 11, 2022
Co-authored-by: Rémy Léone <remy.leone@gmail.com>
Monitob and others added 3 commits August 11, 2022 10:50
Co-authored-by: Rémy Léone <remy.leone@gmail.com>
Co-authored-by: Rémy Léone <remy.leone@gmail.com>
Co-authored-by: Rémy Léone <remy.leone@gmail.com>
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Aug 11, 2022

Codecov Report

Merging #2447 (12af01f) into master (48fa817) will not change coverage.
The diff coverage is 55.00%.

@@           Coverage Diff           @@
##           master    #2447   +/-   ##
=======================================
  Coverage   76.29%   76.29%           
=======================================
  Files         125      125           
  Lines       21328    21328           
=======================================
  Hits        16273    16273           
  Misses       4504     4504           
  Partials      551      551           
Impacted Files Coverage Δ
internal/args/marshal.go 80.55% <ø> (ø)
internal/core/arg_file_content.go 12.82% <0.00%> (ø)
internal/core/autocomplete.go 92.73% <ø> (ø)
internal/core/command_interceptor.go 25.96% <0.00%> (ø)
internal/namespaces/autocomplete/autocomplete.go 11.69% <0.00%> (ø)
internal/namespaces/config/commands.go 78.89% <ø> (ø)
internal/namespaces/instance/v1/custom_image.go 78.90% <ø> (ø)
...nal/namespaces/instance/v1/custom_server_create.go 82.29% <ø> (ø)
internal/tabwriter/tabwriter.go 90.66% <ø> (ø)
internal/namespaces/feedback/issue.go 63.15% <33.33%> (ø)
... and 8 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Monitob Monitob merged commit 8c24f35 into scaleway:master Aug 11, 2022
@Monitob Monitob deleted the fix-linter branch August 11, 2022 09:02
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.

4 participants