Skip to content

chore: fix ci tests again#56

Merged
xiaoshihou514 merged 2 commits intonvimdev:mainfrom
xiaoshihou514:main
Aug 13, 2023
Merged

chore: fix ci tests again#56
xiaoshihou514 merged 2 commits intonvimdev:mainfrom
xiaoshihou514:main

Conversation

@xiaoshihou514
Copy link
Copy Markdown
Member

@xiaoshihou514 xiaoshihou514 commented Aug 13, 2023

a commit changed clang-format's default so the :extra test fails

a commit changed clang-format's default so the :extra test fails??
Comment thread test/filetype_spec.lua
ft('c'):fmt('clang-format'):extra('--verbose'):lint('clang-tidy'):extra('--fix')
same({
cmd = 'clang-format',
args = { '--verbose', '-style=file' },
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm nope this should keep . it should insert extra arguments to default args

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@glepnir
Copy link
Copy Markdown
Member

glepnir commented Aug 13, 2023

looks like #43 remove --stlye=file.

@xiaoshihou514 xiaoshihou514 merged commit 94e1ae6 into nvimdev:main Aug 13, 2023
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.

2 participants