Skip to content

Remove lint action #3

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
Merged
seanbudd merged 22 commits into from
May 21, 2025
Merged

Remove lint action #3

seanbudd merged 22 commits into from
May 21, 2025

Conversation

@RyanMcCleary
Copy link

@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
seanbudd
seanbudd approved these changes May 21, 2025
View reviewed changes
Copy link
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 join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@seanbudd seanbudd seanbudd approved these changes

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