Fix restrictLayoutsTo containment checks for absolute paths#279
Fix restrictLayoutsTo containment checks for absolute paths#279boracomet wants to merge 1 commit intoTryGhost:mainfrom
Conversation
|
You have run out of free Bugbot PR reviews for this billing cycle. This will reset on March 3. To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial. |
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the 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. ✨ Finishing touches🧪 Generate unit tests (beta)
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. Comment |
What this fixes
startsWithpath check incacheLayoutwith apath.relativecontainment check.restrictLayoutsTois relative and the layout path is absolute./allowedvs/allowed-evil).Tests added
restrictLayoutsTois relative.restrictLayoutsToby string prefix.Fixes #270