Skip to content

add new items#2

Open
ashleywolf wants to merge 1 commit intomainfrom
add-items
Open

add new items#2
ashleywolf wants to merge 1 commit intomainfrom
add-items

Conversation

@ashleywolf
Copy link
Copy Markdown
Owner

No description provided.

@github-actions
Copy link
Copy Markdown

⚠️ Contribution Check: Needs Work

This PR has good ideas but needs improvements to meet the guidelines in CONTRIBUTING.md:

Issues to address:

  1. Vague title: "add new items" doesn't clearly explain what's being added

    • ❌ Current: add new items
    • ✅ Better: Add Stale Branch Strudel and CI/CD Cold Brew to menu
  2. Missing description: PR has no body/description

    • Need to add What/Why/Testing sections as shown in CONTRIBUTING.md
    • Explain what items you're adding and why they fit the café theme
  3. Missing tests: New menu items require tests

    • Add tests to tests/test_app.py for both new items
    • See existing tests for examples

Applied label: contribution-check/needs-work

The menu items themselves look great (love the GitHub theme!), but please update the PR with a proper title, description, and tests.

Generated by Contribution Check

@github-actions github-actions bot added the contribution-check/needs-work PR needs improvements before merging label Feb 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution-check/needs-work PR needs improvements before merging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant