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

Fix clang format error #1

Merged
merged 1 commit into from
Sep 20, 2022

Conversation

kevinbackhouse
Copy link

clang-format -i src/easyaccess.cpp

@norbertwg norbertwg merged commit dae3436 into norbertwg:main Sep 20, 2022
@norbertwg
Copy link
Owner

Hello Kevin,
I was already wondering what I should do after I received a report "Clang Format Checker: All jobs have failed" from GitHub. So thanks a lot for this hint. In case I can contribute some more changes, I will consider this.
GitHub webpage suggested to merge the pull request, which I did now here. Is there now anything left for me? As you may have noticed, this is my first contribution. So any hint to do it right (and especially in a way not to cause unneeded work from others) is welcome.
Kind regards
Norbert

@kevinbackhouse
Copy link
Author

@norbertwg: no problem! I think now you just need to add a test file to Exiv2#2362 and it'll be ready to merge. I think you can use this PR as an example of what to do: Exiv2#2337. In other words, add a new file to test/data and a corresponding .out file in test/data/test_reference_files.

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