Skip to content

Refactor .actrc locations and default image survey#600

Merged
cplee merged 2 commits intonektos:masterfrom
catthehacker:xdg-config
Apr 5, 2021
Merged

Refactor .actrc locations and default image survey#600
cplee merged 2 commits intonektos:masterfrom
catthehacker:xdg-config

Conversation

@catthehacker
Copy link
Copy Markdown
Member

Fixes #599

Moved actrc locations to own func since it can be reused.
Moved survey part to own func due to high cyclomatic complexity.
Added XDG config dir to config locations.
Replaced using HOME envvar with go-homedir module since it's already
an indirect dependency and handles home directories better for each GOOS.

CatTheHacker added 2 commits April 3, 2021 15:58
Moved actrc locations to own func since it can be reused.
Moved survey part to own func due to high cyclomatic complexity.
Added XDG config dir to config locations.
Replaced using HOME envvar with go-homedir module since it's already
an indirect dependency and handles home directories better
for each GOOS.
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 3, 2021

Codecov Report

Merging #600 (40ee9bc) into master (b9a7bc6) will decrease coverage by 0.04%.
The diff coverage is 53.84%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #600      +/-   ##
==========================================
- Coverage   49.24%   49.20%   -0.05%     
==========================================
  Files          23       23              
  Lines        2319     2321       +2     
==========================================
  Hits         1142     1142              
- Misses       1051     1052       +1     
- Partials      126      127       +1     
Impacted Files Coverage Δ
pkg/runner/run_context.go 77.55% <53.84%> (-0.46%) ⬇️

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 b9a7bc6...40ee9bc. Read the comment docs.

@catthehacker catthehacker requested a review from cplee April 3, 2021 16:22
Copy link
Copy Markdown
Contributor

@cplee cplee left a comment

Choose a reason for hiding this comment

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

Awesome

@cplee cplee merged commit 4125812 into nektos:master Apr 5, 2021
@catthehacker catthehacker deleted the xdg-config branch May 20, 2021 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enhancement: Follow XDG base directories

2 participants