Skip to content

Refactor GitHub Actions for Python linting and tests - #983

Merged
LexiconCode merged 3 commits into
dictation-toolbox:masterfrom
shailendra-codes:patch-2
Oct 3, 2026
Merged

LexiconCode merged 3 commits into
dictation-toolbox:masterfrom
shailendra-codes:patch-2

Conversation

@shailendra-codes

Copy link
Copy Markdown
Contributor

Title

Description

Related Issue

Motivation and Context

How Has This Been Tested

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue or bug)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Renamed existing command phrases (we discourage this without a strong rationale).

Checklist

  • I have read the CONTRIBUTING document.
  • My code follows the code style of this project.
  • I have checked that my code does not duplicate functionality elsewhere in Caster.
  • I have checked for and utilized existing command phrases from within Caster (delete if not applicable).
  • My code implements all the features I wish to merge in this pull request.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests pass.

Maintainer/Reviewer Checklist

  • Basic functionality has been tested and works as claimed.
  • New documentation is clear and complete.
  • Code is clear and readable.

@LexiconCode LexiconCode left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm not really sure what the point of this PR is outside of a few arbitrary formatting changes. Caster_py is not correct as it breaks the file extension.

@shailendra-codes

Copy link
Copy Markdown
Contributor Author

"Hi @LexiconCode, I have corrected the file extension issue and updated the workflow to use the correct module castervoice. The code changes are verified now. Please review it again. Thanks!"

@LexiconCode

Copy link
Copy Markdown
Member

7
no such option: -u

@shailendra-codes

Copy link
Copy Markdown
Contributor Author

Hi @LexiconCode, I have successfully updated the workflow file (testrunner.yml) at lines 24, 54, and 80. The invalid -u flag has been corrected to the proper -U -r option for pip installation across Linux, Windows, and macOS environments. The PR is now clean and ready for your automated workflow approval. Thanks!

@LexiconCode LexiconCode self-assigned this Oct 3, 2026
@LexiconCode LexiconCode added the CI Issues related to Continuous Integration label Oct 3, 2026
@LexiconCode
LexiconCode merged commit 2b9deab into dictation-toolbox:master Oct 3, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Issues related to Continuous Integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants