Skip to content

fix: use both absolute and relative header paths in header matching #362

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
Jan 1, 2025
Merged

fix: use both absolute and relative header paths in header matching #362

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
907c443
fix: always use absolute (and simplified) header path whenever possible
Tal500 Aug 6, 2024
4e02592
Merge branch 'master' into absolute-path-header
Tal500 Nov 14, 2024
17d74df
respect relative paths, but check both absolute and relative path id …
Nov 14, 2024
f156111
simplify path usage functions to be just strings
Nov 17, 2024
28cdb8a
tidy fix
Nov 17, 2024
2893b68
add integration tests
Dec 10, 2024
196a498
run integraion tests on CI-unixish.yml
Tal500 Dec 15, 2024
52c77b9
Use `pip --user` to avoid macos14 permission issues in CI-unixish.yml
Tal500 Dec 15, 2024
ef7837a
Move the flag `--user` in the pip install row CI-unixish.yml
Tal500 Dec 15, 2024
c2e9b67
Try the flag `--break-system-packages` instead of `--user` in CI-unix…
Tal500 Dec 15, 2024
26ee4d9
Set in a more poratable way break-system-packages to be true CI-unixi…
Tal500 Dec 15, 2024
3b6fc9c
Pytest for Windows in CI-windows.yml
Tal500 Dec 15, 2024
9db5d60
Update testutils.py: allow custom SIMPLECPP_EXE_PATH for integration …
Tal500 Dec 15, 2024
47853fb
Update CI-windows.yml: pass SIMPLECPP_EXE_PATH env var
Tal500 Dec 15, 2024
c026219
Update CI-windows.yml: fix the set command
Tal500 Dec 15, 2024
a402c79
use parametrized tests and improve coverage in integration_test.py
Tal500 Dec 22, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update CI-windows.yml: fix the set command
  • Loading branch information
@Tal500
Tal500 authored Dec 15, 2024
commit c0262191d560e50f4f369a459ece5aadf6072a13
2 changes: 1 addition & 1 deletion .github/workflows/CI-windows.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -62,6 +62,6 @@ jobs:

- name: integration test
run: |
set SIMPLECPP_EXE_PATH=".\${{ matrix.config }}\simplecpp.exe"
set SIMPLECPP_EXE_PATH=.\${{ matrix.config }}\simplecpp.exe
python -m pytest integration_test.py || exit /b !errorlevel!