Skip to content

Default to ~/.config/gmailctl for newly created config dirs. - #455

Merged
mbrt merged 3 commits into
mbrt:masterfrom
lgarron:config-dir
Jun 19, 2026
Merged

Default to ~/.config/gmailctl for newly created config dirs.#455
mbrt merged 3 commits into
mbrt:masterfrom
lgarron:config-dir

Conversation

@lgarron

Copy link
Copy Markdown
Contributor

This is a minimal fix for #144

It is a revision of #393 and uses a library for the internal XDG path calculation per feedback there.

This is a minimal fix for mbrt#144
It is a revision of mbrt#393 and uses a library for the internal XDG path calculation per feedback there.
Comment threadREADME.md
Comment threadcmd/gmailctl/cmd/root_cmd.go Outdated
Comment threadREADME.md Outdated
Co-authored-by: Michele Bertasi <405934+mbrt@users.noreply.github.com>
@codecov

codecovBot commented Jun 18, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 37.56%. Comparing base (1316284) to head (05bc582).
⚠️ Report is 1 commits behind head on master.

Files with missing linesPatch %Lines
cmd/gmailctl/cmd/root_cmd.go0.00%6 Missing ⚠️

❌ Your patch status has failed because the patch coverage (0.00%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@ Coverage Diff @@## master #455 +/- ##
==========================================
- Coverage 37.60% 37.56% -0.05% 
==========================================
Files 54 54 Lines 3670 3674 +4 ==========================================
Hits 1380 1380 - Misses 2199 2203 +4 
Partials 91 91 

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mbrt
mbrt enabled auto-merge (squash) June 19, 2026 06:24
@mbrt
mbrt merged commit b48a982 into mbrt:masterJun 19, 2026
2 checks passed
Sign up for freeto 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

@lgarron@mbrt