Skip to content

fix: The CLI tool uses .gitignore to filter packaged files during deployment.#1334

Open
KaneOne wants to merge 2 commits intoappwrite:masterfrom
KaneOne:patch-1
Open

fix: The CLI tool uses .gitignore to filter packaged files during deployment.#1334
KaneOne wants to merge 2 commits intoappwrite:masterfrom
KaneOne:patch-1

Conversation

@KaneOne
Copy link

@KaneOne KaneOne commented Feb 6, 2026

What does this PR do?

fix: The CLI tool uses .gitignore to filter packaged files during deployment.

Summary by CodeRabbit

New Features

  • Deployment packages now automatically exclude files matching .gitignore patterns, simplifying the packaging process while maintaining existing deployment behavior.

Added support for .gitignore to exclude files during packaging.
Normalize file paths and handle root directory case.
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 6, 2026

Walkthrough

The change adds .gitignore support to the deployment packaging process in the deployment utility. When packaging files, the code now loads patterns from a .gitignore file in the directory being packaged, if present. The tar creation filtering logic is extended to skip files matching these ignore patterns after normalizing paths. The root or current directory receives special handling to remain included. All existing packaging behavior is preserved, including gzip compression and original working directory context.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding .gitignore support to filter files during deployment packaging.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

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.

1 participant

Comments