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.

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

Add support for specifying ignore rules in .clang-format-ignore file using gitignore syntax #87

Merged
merged 12 commits into from
Dec 31, 2023
Merged

Add support for specifying ignore rules in .clang-format-ignore file using gitignore syntax #87

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
58b918f
Add hashes to lockfile
lopopolo Dec 30, 2023
218ac8b
Add ignore package
lopopolo Dec 30, 2023
0d13452
Add support for `.clang-format-ignore`
lopopolo Dec 30, 2023
621905f
Add chalk v5 for colored CLI output, convert package to ESM
lopopolo Dec 30, 2023
77e2b71
Lint with eslint
lopopolo Dec 30, 2023
16c5a35
Ensure ignore file is used on integration tests
lopopolo Dec 30, 2023
1f4f4ba
Remove all 'use strict' pragmas now that package is ESM
lopopolo Dec 30, 2023
c801ec8
Limit concurrency of clang-format invocations with p-limit
lopopolo Dec 30, 2023
857fe24
A lot more refactoring
lopopolo Dec 31, 2023
5fa2dbf
Add option to disable ignore pattern and file processing
lopopolo Dec 31, 2023
40bf525
Increase parallelism of source discovery
lopopolo Dec 31, 2023
ac57499
lint and format
lopopolo Dec 31, 2023
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
6 changes: 6 additions & 0 deletions .clang-format-ignore
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
.git
build
emsdk
node_modules
target
vendor
2 changes: 1 addition & 1 deletion .github/workflows/ci.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ jobs:

- name: Run locally with npx and inferred target directory
if: runner.os != 'Windows'
run: npx -p .. clang-format --check
run: npx -p .. clang-format --check --ignore-path ../.clang-format-ignore
working-directory: "artichoke-integration-test"

js:
Expand Down
8 changes: 2 additions & 6 deletions bin/clang-format.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,9 +1,5 @@
#!/usr/bin/env node

"use strict";
import main from "../src/cli.js";

const main = require("../src/cli");

(async () => {
await main();
})();
await main();
Loading