cva6 icon indicating copy to clipboard operation
cva6 copied to clipboard

[TASK] Integrate lint checking mechanism in GitHub actions

Open fatimasaleem opened this issue 2 years ago • 1 comments

Is there an existing CVA6 task for this?

  • [x] I have searched the existing task issues

Task Description

RTL Linting is used to verify adherence to specified design coding rules that prevent synthesis issues, functional bugs and enforce coding styles for readability & re-use. For that purpose, we have to enable lint warnings and remove/waive them where necessary.

While resolving the current lint warnings, we need to make sure that new design changes are not adding new lint warnings. For this purpose we need to have integrate lint checking mechanism in CI.

Required Changes

Setup CI yml for lint warnings, which should ignore the known current warnings

Current Status

No lint check available

Risks

No response

Prerequisites

No response

KPI (KEY Performance Indicators)

No response

Description of Done

Running lint CI

Associated PRs

No response

fatimasaleem avatar Sep 13 '23 11:09 fatimasaleem

Hello @saadwaheed-10xe @fatimasaleem Big #1513 PR will add a big quantity of code lines. I fear this will add new LINT warnings. I would be pleased to now your recommendation, should you check the LINT regression on this PR ? Do not hesitate to propose.

JeanRochCoulon avatar Oct 10 '23 03:10 JeanRochCoulon