Skip to content

Build workflow#62

Merged
yhouyang02 merged 5 commits intootterfrom
build_workflow
Jan 21, 2026
Merged

Build workflow#62
yhouyang02 merged 5 commits intootterfrom
build_workflow

Conversation

@risha-daz
Copy link
Collaborator

@risha-daz risha-daz commented Jan 21, 2026

  • fix Create .github/workflows/build.yml for running test suite #47
  • Edit workflow for the following tasks:
    • build and install package
    • run all pytest functions
    • run ruff lint check
    • parallel jobs
  • Add "lint" section to pyproject.toml with ruff dependency
  • workflow runs for the following cases:
    • PR into otter or main
    • Merge/push into otter or main

@risha-daz risha-daz added this to the milestone3 milestone Jan 21, 2026
@risha-daz risha-daz self-assigned this Jan 21, 2026
@risha-daz
Copy link
Collaborator Author

risha-daz commented Jan 21, 2026 •
edited
Loading

Note: I suggest we can ignore the lint failure for now for most of our testing. One of us could take the task of fixing the linting errors if possible at a later date. Please check the rest of the tests for now

@risha-daz risha-daz linked an issue Jan 21, 2026 that may be closed by this pull request
Create .github/workflows/build.yml for running test suite #47
Closed
@yhouyang02
Run ruff linter on all python files
ccecdbe 
This commit applies consistent formatting and code style improvements across the wordguess modules and their unit tests. Changes include improved spacing, line breaks, and string quoting, as well as enhanced readability and error message clarity. No functional logic was altered.
@yhouyang02
Ignore F401 in __init__.py for Ruff linting
39eb275 
Adds a per-file ignore for F401 (imported but unused) in __init__.py files to reduce unnecessary linting errors.
@yhouyang02 yhouyang02 merged commit dfe2584 into otter Jan 21, 2026
8 checks passed
@yhouyang02 yhouyang02 deleted the build_workflow branch January 21, 2026 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yhouyang02 yhouyang02 yhouyang02 approved these changes

@sgauth01 sgauth01 Awaiting requested review from sgauth01

@yauchungki1305 yauchungki1305 Awaiting requested review from yauchungki1305

Assignees

@risha-daz risha-daz

Labels

None yet

Projects

None yet

Milestone

milestone3

Development

Successfully merging this pull request may close these issues.

Create .github/workflows/build.yml for running test suite

2 participants

@risha-daz @yhouyang02