Skip to content

Commit fd8c106

Browse files
committed
make lint install and add codespell
1 parent 2bc409f commit fd8c106

File tree

4 files changed

+32
-8
lines changed

4 files changed

+32
-8
lines changed

.github/workflows/ci.yml

Lines changed: 17 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,24 @@ jobs:
2323
. .venv/bin/activate
2424
2525
pip install --upgrade pip
26-
pip install .[dev]
26+
pip install .[lint]
2727
flake8 --statistics .
2828
29+
check_spelling:
30+
name: Check code spelling
31+
runs-on: ubuntu-latest
32+
steps:
33+
- name: Check out Git repository
34+
uses: actions/checkout@v6
35+
- name: Check spelling with codespell
36+
run: |
37+
python3 -m venv .venv
38+
. .venv/bin/activate
39+
40+
pip install --upgrade pip
41+
pip install .[lint]
42+
codespell .
43+
2944
python_ci:
3045
name: Run Python Only
3146
runs-on: ubuntu-latest
@@ -44,7 +59,7 @@ jobs:
4459
mkdir testrun
4560
cd testrun
4661
47-
pytest $GITHUB_WORKSPACE -m "not eda"
62+
pytest $GITHUB_WORKSPACE -n logical -m "not eda"
4863
4964
lint_verilog:
5065
name: Lint Verilog Code

lambdalib/analoglib/la_pll/rtl/la_pll.v

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
*
66
* NOTE: The generic interface was derived by reviewing a number of
77
* publicly open source PLLs and FPGA IP datasheets and via llm
8-
* promting.
8+
* prompting.
99
*
1010
* This is a synthesizable zeroth order PLL model that only
1111
* supports one operating mode: clkout=clkref.
@@ -19,7 +19,7 @@
1919
module la_pll #(parameter PROP = "", // cell property
2020
parameter NIN = 1, // number of input reference clocks
2121
parameter NOUT = 1, // number of output clocks
22-
parameter DIVINW = 8, // iunpyt divider width
22+
parameter DIVINW = 8, // input divider width
2323
parameter DIVFBW = 8, // feedback divider width
2424
parameter DIVOUTW = 8, // output divider width
2525
parameter PHASEW = 8, // phase shift adjust width
@@ -38,7 +38,7 @@ module la_pll #(parameter PROP = "", // cell property
3838
// standard controls
3939
input reset, // active high async reset
4040
input en, // pll enable
41-
input bypass, // pll bypasses
41+
input bypass, // pll bypass
4242
input [NIN-1:0] clksel, // one hot clock selector
4343
input [DIVINW-1:0] divin, // reference divider
4444
input [DIVFBW-1:0] divfb, // feedback divider

lambdalib/fpgalib/la_clb4p0/rtl/la_clb4p0.v

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ module la_clb4p0
8181
if (N>1)
8282
assign fbmux[i][j] = fb[i][cfgfb[(i*K+j)*$clog2(N)+:$clog2(N)]];
8383
else
84-
assign fbmux[i][j] = fb[i][0]; //TODO: suport N=1
84+
assign fbmux[i][j] = fb[i][0]; //TODO: support N=1
8585
end
8686

8787
// select between feedback and primary inputs

pyproject.toml

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,15 +28,19 @@ version = {attr = "lambdalib.__version__"}
2828

2929
[project.optional-dependencies]
3030
# Dev dependencies.
31-
dev = [
32-
"flake8 == 7.3.0",
31+
test = [
3332
"pytest == 8.4.2",
3433
"pytest-xdist==3.8.0",
3534
"pytest-timeout == 2.4.0",
3635
"cocotb == 2.0.1",
3736
"cocotb-bus == 0.3.0"
3837
]
3938

39+
lint = [
40+
"flake8 == 7.3.0",
41+
"codespell == 2.4.1"
42+
]
43+
4044
[tool.pytest.ini_options]
4145
markers = [
4246
"eda: this test requires EDA tools installed to run.",
@@ -48,3 +52,8 @@ testpaths = [
4852
"tests"
4953
]
5054
timeout = "60"
55+
56+
[tool.codespell]
57+
count = true
58+
quiet-level = 3
59+
ignore-words-list = 'inout,inouts,nin,te,dout'

0 commit comments

Comments
 (0)