Skip to content
This repository was archived by the owner on Apr 20, 2026. It is now read-only.

Remove lint action#3

Merged
seanbudd merged 22 commits into
nvaccess:mainfrom
RyanMcCleary:remove-lint-action
May 21, 2025
Merged

Remove lint action#3
seanbudd merged 22 commits into
nvaccess:mainfrom
RyanMcCleary:remove-lint-action

Conversation

@RyanMcCleary
Copy link
Copy Markdown

@RyanMcCleary RyanMcCleary commented May 20, 2025 •
edited by seanbudd
Loading

This commit contains the following changes

  • Removed the lint action, since we are now using pre-commit.
  • Added trailing newlines to some files which did not previously have them.
  • Updated the zip Cargo dependency to 3.0.0 as the 2.x releases have been yanked
  • Added rust files to pre-commit whitespace fixing

RyanMcCleary and others added 18 commits May 14, 2025 22:13
@RyanMcCleary
fixed pyright and ruff errors
6150446
@RyanMcCleary
exclude addon/globalPlugin/MathCAT/yaml/* from ruff linting
27c3bdd
@RyanMcCleary
removed unnecessary noqa comment in Example/test.py
37efe28
@RyanMcCleary
fixed errors in pre-commit config file
538b228
@RyanMcCleary
removed configuration from pyproject.toml that does not apply
8f07b89
@RyanMcCleary
updated maintainer info and changed .yml extension to .yaml
6fee99b
@RyanMcCleary
temporarily commented out no-commit-to-branch hook
25384b9
@RyanMcCleary
updated zip version in Cargo.toml
a20666a
Update translations
3c49323
@RyanMcCleary
added [patch.crates-io] to Cargo.toml
cf156df
@RyanMcCleary
Merge branch 'main' of github.com:RyanMcCleary/MathCATForPython
15fad5a 
Merging
Update translations
31304b2
@RyanMcCleary
added newline at EOF in some files
5fbc8a4
@RyanMcCleary
added newline at end of .rs files
7a2d529
@RyanMcCleary
Merged
4aecb18
@pre-commit-ci
Pre-commit auto-fix
6bff8d7
@RyanMcCleary
removed lint.yaml
984e9b8
@RyanMcCleary
Merge branch 'remove-lint-action' of github.com:RyanMcCleary/MathCATF…
3856957 
…orPython into remove-lint-action

Merging
seanbudd
seanbudd reviewed May 21, 2025
View reviewed changes
Comment thread Cargo.toml Outdated
Comment thread .github/workflows/requirements.txt Outdated
Comment thread build.rs
seanbudd
seanbudd approved these changes May 21, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@seanbudd seanbudd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice thanks!

@seanbudd seanbudd merged commit c73451e into nvaccess:main May 21, 2025
1 check passed
@seanbudd seanbudd mentioned this pull request May 21, 2025
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@seanbudd seanbudd seanbudd approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@RyanMcCleary @seanbudd