Skip to content

fix: write the config file with owner-only permissions - #306

Merged
lucassabreu merged 2 commits into
lucassabreu:mainfrom
nikosavola:nikosavola/push-oksrmmxlsmlt
Sep 17, 2026
Merged

lucassabreu merged 2 commits into
lucassabreu:mainfrom
nikosavola:nikosavola/push-oksrmmxlsmlt

Conversation

@nikosavola

Copy link
Copy Markdown
Contributor
  • the file holds the api token but was created 0644 via viper's default

- the file holds the api token but was created 0644 via viper's default

Co-Authored-By: DeepSeek V4.1 Flash <noreply@deepseek.com>
@nikosavola
nikosavola force-pushed the nikosavola/push-oksrmmxlsmlt branch from bea9283 to 224b268 Compare September 17, 2026 13:48
@codecov

codecov Bot commented Sep 17, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 33.33333% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 32.86%. Comparing base (176547e) to head (fd8e117).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
pkg/cmdutil/config.go 33.33% 2 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #306      +/-   ##
==========================================
+ Coverage   32.46%   32.86%   +0.39%     
==========================================
  Files         141      141              
  Lines        9016     9020       +4     
==========================================
+ Hits         2927     2964      +37     
+ Misses       5974     5932      -42     
- Partials      115      124       +9     
Flag Coverage Δ
unittests 32.86% <33.33%> (+0.39%) ⬆️

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

☔ 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.

@lucassabreu
lucassabreu merged commit 65a7104 into lucassabreu:main Sep 17, 2026
7 of 8 checks passed
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