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

[docs] Add information about clang-format to the automatic code formatting section #3845

Merged

Conversation

diivm
Copy link
Contributor

@diivm diivm commented Apr 2, 2020

closes #3834

@diivm
Copy link
Contributor Author

diivm commented Apr 2, 2020

Suggestions/improvements?

doc/advanced/content/pcl_style_guide.rst Outdated Show resolved Hide resolved
doc/advanced/content/pcl_style_guide.rst Outdated Show resolved Hide resolved
diivm and others added 2 commits April 2, 2020 15:25
Co-Authored-By: Sérgio Agostinho <sergio.r.agostinho@gmail.com>
Copy link
Member

@kunaltyagi kunaltyagi left a comment

Choose a reason for hiding this comment

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

Apart from this, LGTM

doc/advanced/content/pcl_style_guide.rst Outdated Show resolved Hide resolved
@diivm diivm requested a review from kunaltyagi April 2, 2020 13:26
@kunaltyagi kunaltyagi added changelog: fix Meta-information for changelog generation module: docs needs: code review Specify why not closed/merged yet labels Apr 3, 2020
@taketwo taketwo merged commit 10230bc into PointCloudLibrary:master Apr 6, 2020
@SergioRAgostinho SergioRAgostinho removed the needs: code review Specify why not closed/merged yet label Apr 6, 2020
@taketwo taketwo changed the title [docs] update automatic code formatting info to clang-format [docs] Add information about clang-format to the automatic code formatting section May 10, 2020
@diivm diivm deleted the fix/update_style_clang-format branch May 15, 2020 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog: fix Meta-information for changelog generation module: docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[docs] Update Style wrt Code Formatting
4 participants