Skip to content
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.

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

Config files improvements #208

Merged
merged 1 commit into from
Dec 19, 2024
Merged

Config files improvements #208

merged 1 commit into from
Dec 19, 2024

Conversation

ioxid
Copy link
Contributor

@ioxid ioxid commented Dec 16, 2024

.clang-format: preserve include blocks groups (instead of grouping all the includes and sorting them)
.envrc: allow specifying used derivation in environment (mainly so that whoever uses clang could just set PLACEHOLDER_DEVELOP_DERIVATION to .#develop-clang)
.gitignore: add vscode, direnv and cmake related files
CMakeLists.txt: enable tests in root too (so it works without nix develop)
flake.nix: add develop-clang derivation

Copy link

github-actions bot commented Dec 16, 2024

Clang Test Results

  161 files  ±0    161 suites  ±0   16m 41s ⏱️ ±0s
7 086 tests ±0  7 080 ✅ ±0  6 💤 ±0  0 ❌ ±0 
7 172 runs  ±0  7 166 ✅ ±0  6 💤 ±0  0 ❌ ±0 

Results for commit c332aa5. ± Comparison against base commit 01fead9.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Dec 16, 2024

Gcc Test Results

  161 files  ±0    161 suites  ±0   17m 0s ⏱️ ±0s
7 086 tests ±0  7 080 ✅ ±0  6 💤 ±0  0 ❌ ±0 
7 172 runs  ±0  7 166 ✅ ±0  6 💤 ±0  0 ❌ ±0 

Results for commit c332aa5. ± Comparison against base commit 01fead9.

♻️ This comment has been updated with latest results.

@ioxid
Copy link
Contributor Author

ioxid commented Dec 16, 2024

Not sure how the 4 tests got removed. @AndreyMlashkin, any ideas?

@ioxid ioxid added the enhancement New feature or request label Dec 16, 2024
@ioxid ioxid force-pushed the ioxid-config-improvements branch from 42b21a2 to e0a3817 Compare December 16, 2024 11:26
@ioxid ioxid force-pushed the ioxid-config-improvements branch from 5a03cc8 to c332aa5 Compare December 19, 2024 09:25
@AndreyMlashkin AndreyMlashkin self-requested a review December 19, 2024 09:28
@ioxid ioxid enabled auto-merge December 19, 2024 09:30
@ioxid ioxid added this pull request to the merge queue Dec 19, 2024
Merged via the queue into master with commit 9b3fb27 Dec 19, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants