Skip to content

Add format target for source code formatting - #2

Open
kgugala wants to merge 1 commit into
verilog-to-routing:masterfrom
antmicro:code-format
Open

Add format target for source code formatting#2
kgugala wants to merge 1 commit into
verilog-to-routing:masterfrom
antmicro:code-format

Conversation

@kgugala

@kgugala kgugala commented Mar 27, 2019

Copy link
Copy Markdown

This PR adds make format target which can be used to autoformat the code in the project. The target uses clang-format-5.0 tool.

Signed-off-by: Lukasz Dalek <ldalek@antmicro.com>
@mithro

mithro commented Mar 27, 2019

Copy link
Copy Markdown

@kgugala Can you link to the output were the code has been formatted?

Could you also add instructions for @kmurray on how to do the formatting after this pull request lands?

@kgugala

kgugala commented Mar 27, 2019

Copy link
Copy Markdown
Author

the results can be found here: https://github.com/antmicro/libblifparse/tree/run-format
to run the format do:

mkdir build && cd build
cmake ..
make format

@vaughnbetz

Copy link
Copy Markdown
Contributor

@soheilshahrouz : we should probably turn on format checking for this library too. I didn't notice this stray PR until now. Can you take a look and see if we should merge or update?

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.

3 participants