Skip to content

Ignore tests#9

Merged
sfroning88 merged 2 commits into
mainfrom
ignore-tests
Jun 3, 2026
Merged

Ignore tests#9
sfroning88 merged 2 commits into
mainfrom
ignore-tests

Conversation

@sfroning88
Copy link
Copy Markdown
Owner

@sfroning88 sfroning88 commented Jun 3, 2026

Python Nurse Pull Request

Changes

  • ignore /tests/** and /e2e/**

Summary by CodeRabbit

  • Chores
    • Package version updated to 1.0.6.
    • Build script enhanced to filter out test and e2e directories from change detection reports.

@sfroning88 sfroning88 self-assigned this Jun 3, 2026
@sfroning88 sfroning88 added the enhancement New feature or request label Jun 3, 2026
@sfroning88 sfroning88 merged commit 3317a6d into main Jun 3, 2026
1 check was pending
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 3, 2026

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ca22b860-7503-4207-a5ce-f8ff47d8411c

📥 Commits

Reviewing files that changed from the base of the PR and between 9fa5c99 and ff8c60b.

📒 Files selected for processing (2)
  • package.json
  • scripts/get-changed-files.sh

📝 Walkthrough

Walkthrough

This PR increments the package version from 1.0.3 to 1.0.6 and adds directory filtering to the get-changed-files.sh script. The script now defines and calls a helper function that removes file paths under tests/ and e2e/ directories from the per-extension change lists, logging the count of excluded entries when applicable.

Changes

Version and Script Update

Layer / File(s) Summary
Package version update
package.json
Version field bumped from 1.0.3 to 1.0.6.
Directory filtering in get-changed-files script
scripts/get-changed-files.sh
New filter_excluded_dirs helper function filters out tests/ and e2e/ paths from raw file lists via regex, rewrites the input file in-place, and logs exclusion counts. The main script loop calls this filter after deduplication and sorting for each file extension (.py, .sql, .md).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A version springs forward, from three-point-oh-three,
Scripts now filter tests away so cleanly,
No e2e clutter in the changed file dance,
Version 1.0.6 gets its second chance! ✨

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ignore-tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sfroning88 sfroning88 deleted the ignore-tests branch June 3, 2026 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant