search

chipsalliance / Cores-VeeR-EL2

VeeR EL2 Core
https://chipsalliance.github.io/Cores-VeeR-EL2/html/
Apache License 2.0
246 stars 74 forks source link

remove verible formatter #203

Closed wsipak closed 2 months ago

wsipak commented 3 months ago

The current codebase is not formatted properly to make good use of the formatter. The suggestions we're getting in PRs result in inconsistent style and indentation. Running the formatter on the current codebase results in:

50 files changed, 19419 insertions(+), 15792 deletions(-)

I think we should first reformat the whole code and then reintroduce the formatter.

We still have the linter enabled, so we're not completely getting rid of Verible here.

github-actions[bot] commented 3 months ago

Links to coverage and verification reports for this PR (#203) are available at https://chipsalliance.github.io/Cores-VeeR-EL2/