Move build system to UV, add vscode debugger/test ui, add python 3.13 build/test target#188
Open
nateinaction wants to merge 18 commits intoeconchick:masterfrom
Open
Move build system to UV, add vscode debugger/test ui, add python 3.13 build/test target#188nateinaction wants to merge 18 commits intoeconchick:masterfrom
nateinaction wants to merge 18 commits intoeconchick:masterfrom
Conversation
Author
|
Happy Monday @econchick! I've updated the PR to use the |
nateinaction
commented
Jul 7, 2025
Comment on lines
+238
to
+245
| current_results = [] | ||
| for t in texts: | ||
| if ( | ||
| t.hasAttribute("data-interrogate") | ||
| and t.childNodes | ||
| and hasattr(t.childNodes[0], "data") | ||
| ): | ||
| current_results.append(t.childNodes[0].data) |
Author
There was a problem hiding this comment.
teensy weensy code change to satisfy mypy
nateinaction
commented
Jul 7, 2025
| try: | ||
| import cairosvg | ||
| except ImportError: # pragma: no cover | ||
| except (ImportError, OSError): # pragma: no cover |
Author
There was a problem hiding this comment.
Another teensy weensy code change to prevent an OSError from ruining the day while running tests
E OSError: no library called "cairo-2" was found
E no library called "cairo" was found
E no library called "libcairo-2" was found
nateinaction
commented
Jul 7, 2025
Comment on lines
+253
to
+258
| try: | ||
| parsed_tree = ast.parse(source_tree) | ||
| except SyntaxError as e: | ||
| click.echo(f"Failed to parse {filename}: {str(e)}", err=True) | ||
| return None | ||
|
|
Author
There was a problem hiding this comment.
Improved error message when interrogate encounters a syntax error in a codebase
Before
After
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hey Lynn,
I playfully (🪂) worked my way toward debugging issue #187. To start that journey I wanted to update some of the developer tooling to help me get to debugging a little quicker. I also needed to update the CI system to get builds passing since some of the action images had been deprecated and removed. What do you think?
uvbased commandsbadge_gen.pyto satisfy mypybadge_gen.pyto prevent anOSErrorfrom ruining the dayBefore
After
Have you tested this? If so, how?
All tests running in CI and I have tested all contributor doc commands locally.
Checklist for PR author(s)
tox -e docsorinterrogate -c pyproject.toml(I mean, we should set a good example 😄).README.rst.README.rstanddocs/index.rstfor any new/changed CLI flags.versionadded,versionchanged, ordeprecateddirectives. Find the appropriate next version in the project's__init__.pyfile.Release note
Solves #187