Skip to content

test: add smoke import tests for app package - #18

Merged
razinkele merged 1 commit into
mainfrom
test/add-app-import-test
Jan 4, 2026
Merged

razinkele merged 1 commit into
mainfrom
test/add-app-import-test

Conversation

@razinkele

Copy link
Copy Markdown
Owner

Add small smoke tests that import �pp.app and �pp.logger so import-time errors (like syntax errors or package/script CWD issues) fail CI early and get caught automatically.

This is a focused, low-risk test intended to prevent regressions similar to the ImportError/SyntaxError we just fixed.

@razinkele razinkele added the auto-fix Automated, safe formatting or small fixes label Jan 4, 2026
@coderabbitai

coderabbitai Bot commented Jan 4, 2026

Copy link
Copy Markdown

Warning

Rate limit exceeded

@razinkele has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 9 minutes and 18 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 6a5e377 and 4b25579.

📒 Files selected for processing (1)
  • tests/test_app_import.py

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.

@github-actions

github-actions Bot commented Jan 4, 2026

Copy link
Copy Markdown

CI Auto-Fix ran for workflow run 20698551962: formatting fixes applied or diagnostic PR created.

@razinkele
razinkele merged commit 17e77f7 into main Jan 4, 2026
1 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-fix Automated, safe formatting or small fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant