Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

cleanup codebase using pre-commit checks: ruff, codespell #42

Merged
merged 27 commits into from
Jan 4, 2025
Merged

cleanup codebase using pre-commit checks: ruff, codespell #42

merged 27 commits into from
Jan 4, 2025

Conversation

alexlancaster
Copy link
Member

  • checks syntax with ruff

  • spelling with codespell

  • markdown with mdformat

@alexlancaster
re-enable filterwarnings to include error
aa16ab0
@alexlancaster
add ruff configuration and update simple fixes
f30bc7d
@alexlancaster
pre-commit ci hook
754ff02
@alexlancaster
set minimum target for ruff as py39
5ee9dd5
@alexlancaster
add markdown check to pre-commit
3b84356
@alexlancaster
add spell check to pre-commit
b18eb0b
@alexlancaster
add mypy for type checking
9b75df1
@alexlancaster
add mypy config to pyproject
f0510e8
@alexlancaster
add more pre-commit checks
a1f644f 
* run Markdown checker
@alexlancaster
tidy paper.md
617c165
@alexlancaster
move ignore-words-list into config
edfb9ae
@alexlancaster
remove mypy type checking
48c85ac 
* not using for the time being
@alexlancaster
Merge remote-tracking branch 'origin/main' into repo_review_updated
271cafe
@alexlancaster
update manage_graphdb module with ruff changes
d783e7f
@alexlancaster
update parser_scribl module as per ruff
8b90667
@alexlancaster
ruff updates for process_graphdb_data module
9e1444b
@alexlancaster
ruff changes for process_zotero module
5dd3dad
@alexlancaster
ruff fixes for scribl script
8c170be
@alexlancaster
revert open() -> Path.open():
f210f38 
* not supported in older Python <= 3.10
@alexlancaster
revert os.remove -> Path.unlink
c932a23
@alexlancaster
ruff fixes for unit tests for manage_graphdb
20ebc8c
@alexlancaster
ruff remaining unit tests
f816f15 
* need to still update `os.cwd` and `os.path.getsize`
@alexlancaster
add .gitignore for scribl/_version.py
9af85af
@alexlancaster
os.getcwd() -> Path.cwd()
f24f385
@alexlancaster
use Path for stat() size
4f8551f
@alexlancaster
add shell to snippets
576cca6
@alexlancaster alexlancaster added the python Pull requests that update Python code label Jan 4, 2025
@alexlancaster alexlancaster self-assigned this Jan 4, 2025
@alexlancaster alexlancaster changed the title add more pre-commit checks: ruff, codespell cleanup codebase using pre-commit checks: ruff, codespell Jan 4, 2025
@alexlancaster alexlancaster merged commit 1e1f10f into main Jan 4, 2025
44 checks passed
@alexlancaster alexlancaster deleted the repo_review_updated branch January 4, 2025 07:52
@alexlancaster alexlancaster added the dependencies Pull requests that update a dependency file label Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@alexlancaster alexlancaster

Labels
dependencies Pull requests that update a dependency file python Pull requests that update Python code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@alexlancaster