Don't skip reading the local repo when `--repo` specifies a repo slug #398

Merged

6543 merged 3 commits from noerw/tea:fix/378-repoflag-context into master 1 month ago

noerw commented 1 month ago

I added this check in #327, but it wasn't needed at all
as the error case it intended to catch where already handled by checking if the path exists.

fixes #378

noerw added this to the v0.8.0 milestone 1 month ago

noerw added the

kind/bug

label 1 month ago

noerw added 1 commit 1 month ago

21171731e7 dont skip reading local repo when repo flag specifies a repo slug

the check was added in #327, but isnt needed, as the error case is already handled by the path exists check

6543 approved these changes 1 month ago

6543 commented 1 month ago

Collaborator

in combination with #393 - it fixes the issue

6543 added 1 commit 1 month ago

853786c549 Merge branch 'master' into fix/378-repoflag-context

6543 added 1 commit 1 month ago

30a71b7685 Merge branch 'master' into fix/378-repoflag-context

zeripath approved these changes 1 month ago

6543 merged commit 1731e00ebd into master 1 month ago

6543 referenced this issue from a commit 1 month ago

Don't skip reading the local repo when `--repo` specifies a repo slug (#398)

6543 deleted branch fix/378-repoflag-context 1 month ago

Reviewers

6543 approved these changes 1 month ago

zeripath approved these changes 1 month ago

continuous-integration/drone/pr Build is passing

Details

The pull request has been merged as 1731e00ebd.

Sign in to join this conversation.

No Assignees

3 Participants

Notifications

Due Date

No due date set.