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

Add formatting job on CI #3420

Merged
merged 4 commits into from
Oct 17, 2019
Merged

Conversation

taketwo
Copy link
Member

@taketwo taketwo commented Oct 16, 2019

This adds a new Azure pipeline that checks code formatting.

Additionally, as proposed in #3325:

  • remove modules that are not yet formatted from the formatting whitelist;
  • lift clang-format required version to 7.

Note that this pipeline won't be run on Azure until it's configuration is updated through web interface. This, however, can be done only after this PR is merged and pipeline YAML file is present in the repository. For reference, here is the result of running this pipeline on my fork: https://dev.azure.com/u7p11/pcl/_build/results?buildId=702

Closes #3325.

@taketwo taketwo merged commit a8f6435 into PointCloudLibrary:master Oct 17, 2019
@taketwo taketwo deleted the add-formatting-job branch October 17, 2019 07:41
@taketwo
Copy link
Member Author

taketwo commented Oct 17, 2019

New pipeline: https://dev.azure.com/PointCloudLibrary/pcl/_build?definitionId=6&_a=summary

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create a CI job that checks formatting
2 participants