Skip to content

Resolve .gitattributes conflict markers and remove unused generator table - #143

Merged
timkpaine merged 2 commits into
mainfrom
fix-gitattributes-conflict
Nov 14, 2026
Merged

timkpaine merged 2 commits into
mainfrom
fix-gitattributes-conflict

Conversation

@timkpaine

Copy link
Copy Markdown
Member

Description

  • .gitattributes contained both sides of a merge conflict from a 2025 Copier update. It now matches the template: line ending rules for .md, .py, .toml, and .yaml.
  • DefaultMSVCGenerator in hatch_cpp/toolchains/cmake.py mapped MSVC toolset versions to CMake generator names. It was added with the vcpkg toolchain work and never read. Since Run tests on macOS and Windows and fix the Windows failures #142 CMake picks the generator itself, and the table already lacked Visual Studio 2026, so it is removed.

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactor / code cleanup
  • CI / build configuration
  • Other (describe below)

Checklist

  • Linting passes (make lint)
  • Tests pass (make test)
  • New tests added for new functionality
  • Documentation updated (if applicable)
  • Changelog / version bump (if applicable)

A Copier update in 2025 committed both sides of a conflict. Keep the template's per-extension line ending rules.

Signed-off-by: Tim Paine <3105306+timkpaine@users.noreply.github.com>
The table mapped MSVC toolset versions to CMake generator names. It was added with the vcpkg toolchain work and never read. CMake now picks the generator itself, and the table already lacked Visual Studio 2026.

Signed-off-by: Tim Paine <3105306+timkpaine@users.noreply.github.com>
@github-actions

github-actions Bot commented Nov 14, 2026 •

Copy link
Copy Markdown
Contributor

Test Results

110 tests  ±0   110 ✅ ±0   1m 21s ⏱️ +11s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 0a4715f. ± Comparison against base commit 36e84e2.

♻️ This comment has been updated with latest results.

@codecov

codecov Bot commented Nov 14, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.71%. Comparing base (36e84e2) to head (0a4715f).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #143      +/-   ##
==========================================
- Coverage   97.71%   97.71%   -0.01%     
==========================================
  Files          16       16              
  Lines        1578     1577       -1     
==========================================
- Hits         1542     1541       -1     
  Misses         36       36              

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

@timkpaine
timkpaine merged commit c3717b9 into main Nov 14, 2026
9 checks passed
@timkpaine
timkpaine deleted the fix-gitattributes-conflict branch November 14, 2026 00:52
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.

1 participant