Skip to content

Commit 910e60a

Browse files
committed
fix(ci): configure mypy to resolve path ambiguity
1 parent 28ddd5d commit 910e60a

2 files changed

Lines changed: 43 additions & 40 deletions

File tree

coverage.xml

Lines changed: 38 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
<?xml version="1.0" ?>
2-
<coverage version="7.8.2" timestamp="1749702825638" lines-valid="807" lines-covered="697" line-rate="0.8637" branches-valid="400" branches-covered="334" branch-rate="0.835" complexity="0">
2+
<coverage version="7.8.2" timestamp="1749785619506" lines-valid="809" lines-covered="699" line-rate="0.864" branches-valid="400" branches-covered="334" branch-rate="0.835" complexity="0">
33
<!-- Generated by coverage.py: https://coverage.readthedocs.io/en/7.8.2 -->
44
<!-- Based on https://raw.githubusercontent.com/cobertura/web/master/htdocs/xml/coverage-04.dtd -->
55
<sources>
66
<source>/Users/shouryamaheshwari/Desktop/Projects/ContextCraft/src/contextcraft</source>
77
</sources>
88
<packages>
9-
<package name="." line-rate="0.7473" branch-rate="0.5833" complexity="0">
9+
<package name="." line-rate="0.7527" branch-rate="0.5833" complexity="0">
1010
<classes>
1111
<class name="__init__.py" filename="__init__.py" complexity="0" line-rate="1" branch-rate="1">
1212
<methods/>
@@ -16,7 +16,7 @@
1616
<line number="9" hits="1"/>
1717
</lines>
1818
</class>
19-
<class name="main.py" filename="main.py" complexity="0" line-rate="0.7386" branch-rate="0.5833">
19+
<class name="main.py" filename="main.py" complexity="0" line-rate="0.7444" branch-rate="0.5833">
2020
<methods/>
2121
<lines>
2222
<line number="14" hits="1"/>
@@ -63,50 +63,52 @@
6363
<line number="178" hits="1"/>
6464
<line number="179" hits="1"/>
6565
<line number="180" hits="1"/>
66-
<line number="183" hits="1"/>
66+
<line number="181" hits="1"/>
6767
<line number="184" hits="1"/>
68-
<line number="243" hits="1"/>
69-
<line number="245" hits="1"/>
70-
<line number="246" hits="1" branch="true" condition-coverage="100% (2/2)"/>
71-
<line number="247" hits="1"/>
72-
<line number="248" hits="1" branch="true" condition-coverage="100% (2/2)"/>
73-
<line number="249" hits="1"/>
74-
<line number="250" hits="1" branch="true" condition-coverage="50% (1/2)" missing-branches="254"/>
75-
<line number="251" hits="1"/>
68+
<line number="185" hits="1"/>
69+
<line number="244" hits="1"/>
70+
<line number="246" hits="1"/>
71+
<line number="247" hits="1" branch="true" condition-coverage="100% (2/2)"/>
72+
<line number="248" hits="1"/>
73+
<line number="249" hits="1" branch="true" condition-coverage="100% (2/2)"/>
74+
<line number="250" hits="1"/>
75+
<line number="251" hits="1" branch="true" condition-coverage="50% (1/2)" missing-branches="255"/>
7676
<line number="252" hits="1"/>
77-
<line number="254" hits="0"/>
78-
<line number="261" hits="1"/>
79-
<line number="264" hits="1"/>
80-
<line number="266" hits="1"/>
81-
<line number="267" hits="1" branch="true" condition-coverage="50% (1/2)" missing-branches="268"/>
82-
<line number="268" hits="0"/>
77+
<line number="253" hits="1"/>
78+
<line number="255" hits="0"/>
79+
<line number="262" hits="1"/>
80+
<line number="265" hits="1"/>
81+
<line number="267" hits="1"/>
82+
<line number="268" hits="1" branch="true" condition-coverage="50% (1/2)" missing-branches="269"/>
8383
<line number="269" hits="0"/>
84-
<line number="271" hits="1"/>
84+
<line number="270" hits="0"/>
8585
<line number="272" hits="1"/>
86-
<line number="279" hits="1"/>
86+
<line number="273" hits="1"/>
8787
<line number="280" hits="1"/>
8888
<line number="281" hits="1"/>
8989
<line number="282" hits="1"/>
9090
<line number="283" hits="1"/>
91-
<line number="286" hits="1"/>
92-
<line number="287" hits="1"/>
93-
<line number="324" hits="0"/>
91+
<line number="284" hits="1"/>
92+
<line number="285" hits="1"/>
93+
<line number="288" hits="1"/>
94+
<line number="289" hits="1"/>
9495
<line number="326" hits="0"/>
95-
<line number="327" hits="0" branch="true" condition-coverage="0% (0/2)" missing-branches="328,329"/>
9696
<line number="328" hits="0"/>
97-
<line number="329" hits="0" branch="true" condition-coverage="0% (0/2)" missing-branches="331,342"/>
98-
<line number="331" hits="0"/>
99-
<line number="332" hits="0" branch="true" condition-coverage="0% (0/2)" missing-branches="333,336"/>
97+
<line number="329" hits="0" branch="true" condition-coverage="0% (0/2)" missing-branches="330,331"/>
98+
<line number="330" hits="0"/>
99+
<line number="331" hits="0" branch="true" condition-coverage="0% (0/2)" missing-branches="333,344"/>
100100
<line number="333" hits="0"/>
101-
<line number="334" hits="0"/>
101+
<line number="334" hits="0" branch="true" condition-coverage="0% (0/2)" missing-branches="335,338"/>
102+
<line number="335" hits="0"/>
102103
<line number="336" hits="0"/>
103-
<line number="342" hits="0"/>
104-
<line number="343" hits="0"/>
105-
<line number="347" hits="0"/>
106-
<line number="348" hits="0"/>
104+
<line number="338" hits="0"/>
105+
<line number="344" hits="0"/>
106+
<line number="345" hits="0"/>
107107
<line number="349" hits="0"/>
108108
<line number="350" hits="0"/>
109109
<line number="351" hits="0"/>
110+
<line number="352" hits="0"/>
111+
<line number="353" hits="0"/>
110112
</lines>
111113
</class>
112114
</classes>
@@ -563,10 +565,10 @@
563565
<line number="402" hits="0"/>
564566
<line number="403" hits="0"/>
565567
<line number="404" hits="0"/>
566-
<line number="406" hits="1"/>
567-
<line number="410" hits="1" branch="true" condition-coverage="50% (1/2)" missing-branches="exit"/>
568-
<line number="411" hits="1"/>
569-
<line number="414" hits="1"/>
568+
<line number="407" hits="1"/>
569+
<line number="411" hits="1" branch="true" condition-coverage="50% (1/2)" missing-branches="exit"/>
570+
<line number="412" hits="1"/>
571+
<line number="415" hits="1"/>
570572
</lines>
571573
</class>
572574
<class name="tree_generator.py" filename="tools/tree_generator.py" complexity="0" line-rate="0.8641" branch-rate="0.9474">

pyproject.toml

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ quote-style = "double"
3939
indent-style = "space"
4040

4141
[tool.ruff.lint]
42-
# Add any specific lint rule configurations here if needed
42+
select = ["E", "W", "F", "I", "B", "C4", "N", "UP", "S", "D"]
4343

4444
[tool.ruff.lint.per-file-ignores]
4545
"src/contextcraft/main.py" = ["E501"]
@@ -53,9 +53,9 @@ ignore_missing_imports = true
5353
explicit_package_bases = true
5454
mypy_path = ["src"] # Changed to a list format
5555
namespace_packages = true # Added to help with package discovery
56-
# Optional, but can be more explicit for what to check:
57-
files = ["contextcraft", "tests"] # Check these specific packages/dirs
58-
# packages = ["contextcraft"] # Check this specific package (when src is on path)
56+
strict = true
57+
exclude = ["tests/"]
58+
packages = ["src.contextcraft"]
5959

6060
# Configuration for Pytest (optional, many things are auto-discovered)
6161
[tool.pytest.ini_options]
@@ -80,6 +80,7 @@ show_missing = true
8080
# Refer to Bandit documentation for available test IDs:
8181
# https://bandit.readthedocs.io/en/latest/plugins/index.html
8282
skips = ["B101"] # Skip assert_used since we use asserts in tests
83+
exclude_dirs = ["tests"]
8384
# tests = [] # To run only specific tests
8485
# Example: Set confidence level to MEDIUM or HIGH to reduce noise from LOW findings.
8586
# severity = ["MEDIUM", "HIGH"]

0 commit comments

Comments
 (0)