Skip to content

feat: add justfile#11

Merged
gemmadanks merged 4 commits intomainfrom
feat-add-justfile
Oct 11, 2025
Merged

feat: add justfile#11
gemmadanks merged 4 commits intomainfrom
feat-add-justfile

Conversation

@gemmadanks
Copy link
Copy Markdown
Owner

@gemmadanks gemmadanks commented Oct 11, 2025

Recipes for running tests, pre-commit hooks,

Prerequisites checklist

Please check each of the following before merging:

  • I have updated any relevant documentation
  • I have added unit tests for any new code
  • I have linked to any relevant issues
  • I will squash my commits and, where needed, edit the commit messages to make them more informative follow the conventional commits format
  • This pull request is ready to merge (leave unchecked to indicate a WIP)

What is the purpose of this pull request?

Check all that apply:

  • Documentation update
  • Bug fix
  • New feature
  • Changes to an existing feature
  • Refactor
  • Increasing test coverage
  • Other, please explain:

Recipes for running tests, pre-commit hooks,
Copilot AI review requested due to automatic review settings October 11, 2025 19:13
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a justfile to provide convenient developer commands for Poetry-based projects, replacing the need to remember lengthy poetry and pre-commit commands with simple just <command> aliases.

  • Introduces a comprehensive justfile with recipes for testing, linting, formatting, documentation, and pre-commit operations
  • Updates README.md to document the new justfile commands and replace old command examples
  • Removes an erroneous line from the test file

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
justfile Adds new justfile with 15 developer command recipes for common tasks like testing, linting, and documentation
README.md Updates documentation to reference justfile commands and adds justfile to project structure
tests/unit/test_greet.py Removes misplaced test parameter line
Comments suppressed due to low confidence (1)

tests/unit/test_greet.py:8

  • This line appears to be test parameters for test_say_hello but is incorrectly placed after test_say_goodbye. The parameters reference 'Hello' greetings but are positioned after a goodbye test function.
    "input_name,expected_output", [("Alice", "Hello, Alice!"), ("Bob", "Hello, Bob!")]

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 11, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Copilot AI review requested due to automatic review settings October 11, 2025 19:40
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings October 11, 2025 19:42
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@gemmadanks gemmadanks merged commit 8b2396a into main Oct 11, 2025
25 checks passed
@gemmadanks gemmadanks deleted the feat-add-justfile branch October 11, 2025 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants