Skip to content

Conversation

@phm07
Copy link
Contributor

@phm07 phm07 commented Nov 6, 2025

The path package only supports slash-separated paths. We should use filepath for platform-specific path handling instead.

Fixes #1225

@phm07 phm07 self-assigned this Nov 6, 2025
@phm07 phm07 requested a review from a team as a code owner November 6, 2025 12:27
@phm07 phm07 added the bug label Nov 6, 2025
@codecov
Copy link

codecov bot commented Nov 6, 2025

Codecov Report

❌ Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.28%. Comparing base (f21f16b) to head (128a30d).
⚠️ Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
internal/scripts/generate_docs.go 0.00% 5 Missing ⚠️
internal/state/config/config.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1229   +/-   ##
=======================================
  Coverage   72.28%   72.28%           
=======================================
  Files         311      311           
  Lines       11407    11407           
=======================================
  Hits         8245     8245           
  Misses       2216     2216           
  Partials      946      946           
Flag Coverage Δ
e2e 51.46% <0.00%> (ø)
unit 67.15% <0.00%> (ø)

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

☔ View full report in Codecov by Sentry.
📢 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.

@jooola
Copy link
Member

jooola commented Nov 7, 2025

Should we run some tests in windows?

@phm07
Copy link
Contributor Author

phm07 commented Nov 7, 2025

Should we run some tests in windows?

Yeah, I think that would make sense

@phm07 phm07 merged commit 3cbb041 into main Nov 7, 2025
7 checks passed
@phm07 phm07 deleted the fix-windows-path branch November 7, 2025 13:29
jooola pushed a commit that referenced this pull request Nov 7, 2025
<!-- section-start changelog -->
### Features

- format user provided TXT records when not quoted (#1208)

### Bug Fixes

- more readable default time format (#1197)
- **iso**: broken `--type` flag in list command (#1221)
- Storage Boxes not listed in `hcloud all list` (#1222)
- consistent tab spacing in describe commands (#1216)
- filepaths not correctly resolved on Windows (#1229)

<!-- section-end changelog -->

---

<details>
<summary><h4>PR by <a
href="https://github.com/apricote/releaser-pleaser">releaser-pleaser</a>
🤖</h4></summary>

If you want to modify the proposed release, add you overrides here. You
can learn more about the options in the docs.

## Release Notes

### Prefix / Start

This will be added to the start of the release notes.

~~~~rp-prefix
~~~~

### Suffix / End

This will be added to the end of the release notes.

~~~~rp-suffix
~~~~

</details>

Co-authored-by: Hetzner Cloud Bot <>
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.

Config file path not as documented on Windows, error on hcloud context create

3 participants