--repo flag skips reading the local repo #378

New Issue

noerw · 2021-07-08T20:19:29Z

noerw commented

2021-07-08 20:19:29 +00:00

Commands that require a local repo (such as tea pr create), don't work when an override for the remote repo is specified:

$ tea pr create --repo foo/bar # goal: create a PR in a different repo

Local repository required: Execute from a repo dir, or specify a path with --repo.

ref #377 (comment)

that said, tea pr create should still work without a local repo. local repo should just be used to find a default for the head branch when available

Commands that require a local repo (such as `tea pr create`), don't work when an override for the remote repo is specified: ``` $ tea pr create --repo foo/bar # goal: create a PR in a different repo Local repository required: Execute from a repo dir, or specify a path with --repo. ``` ref https://gitea.com/gitea/tea/issues/377#issuecomment-441849 --- that said, `tea pr create` should still work without a local repo. local repo should just be used to find a default for the head branch when available

noerw added the

kind/bug

label 2021-07-08 20:19:29 +00:00

noerw added this to the v0.8.0 milestone 2021-07-08 20:19:33 +00:00

noerw referenced this issue

2021-07-08 22:06:53 +00:00

difficulty creating PR #377

noerw referenced this issue

2021-07-09 18:25:00 +00:00

drop features to improve usability #379

noerw referenced this issue

2021-08-25 08:30:13 +00:00

tea pr create: make local repo optional #393

noerw commented

2021-08-31 08:16:39 +00:00

the specific issue with tea pr create has been adressed.

the remaining issue here is caused by a change from #327:

		modules/context/context.go
		Line 102 in 3fca309f2c
	
					if len(repoFlag) == 0 || repoFlagPathExists {

Not sure what this was about, need to check the testcases in #320 without this if statement..

the specific issue with `tea pr create` has been adressed. the remaining issue here is caused by a change from #327: https://gitea.com/gitea/tea/src/commit/3fca309f2c60045ccb2798f84d1314684783eace/modules/context/context.go#L102 Not sure what this was about, need to check the testcases in #320 without this if statement..

noerw referenced a pull request that will close this issue

2021-09-12 09:35:36 +00:00

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

6543 referenced a pull request that will close this issue

2021-09-22 00:38:54 +00:00

tea pr create: make local repo optional #393

6543 closed this issue

2021-09-22 15:48:21 +00:00

6543 referenced this issue from a commit

2021-09-22 15:48:22 +00:00

tea pr create: make local repo optional (#393)

6543 reopened this issue