240139 2022-05-05 13:52:57 -0700 [git-webkit] Automatically file bug 2022-05-24 00:00:38 -0700 1 1 1 Unclassified WebKit Tools / Tests WebKit Local Build Unspecified Unspecified RESOLVED FIXED InRadar P2 Normal --- 1 jbedard jbedard rego webkit-bug-importer oldest_to_newest 1866979 0 jbedard 2022-05-05 13:52:57 -0700 git-webkit should allow callers to automatically create a bug if no bug URL is provided and project defines an issue tracker. 1866981 1 webkit-bug-importer 2022-05-05 13:57:26 -0700 <rdar://problem/92818331> 1867026 2 jbedard 2022-05-05 16:04:39 -0700 Pull request: https://github.com/WebKit/WebKit/pull/531 1867063 3 ews-feeder 2022-05-05 17:49:40 -0700 Committed r293875 (250333@main): <https://commits.webkit.org/250333@main> Reviewed commits have been landed. Closing PR #531 and removing active labels. 1870188 4 jbedard 2022-05-18 15:44:26 -0700 *** Bug 240339 has been marked as a duplicate of this bug. *** 1870335 5 rego 2022-05-19 04:51:34 -0700 How does this work? For example I have a local commit with Need the bug URL (OOPS!). Then I do "Tools/Scripts/git-webkit upload" and nobody asks to open a new bug, and a new bug is not opened. I believe with "webkit-patch" that worked automatically. Thanks. 1870445 6 jbedard 2022-05-19 10:26:47 -0700 (In reply to Manuel Rego Casasnovas from comment #5) > How does this work? > > For example I have a local commit with > Need the bug URL (OOPS!). > > Then I do "Tools/Scripts/git-webkit upload" and nobody asks to open a new > bug, and a new bug is not opened. > > I believe with "webkit-patch" that worked automatically. > > Thanks. Can you share a little more about what branch your commit is on? And is this a commit created by `git-webkit upload`, or is it one you created with `git commit`. This behavior is currently tied to the creation of a PR branch, `git-webkit branch` while main is checked out is the more narrow example. As a side note, I wonder if `git-webkit commit -i <bug title> should also trigger bug creation.... 1870502 7 rego 2022-05-19 12:39:38 -0700 (In reply to Jonathan Bedard from comment #6) > Can you share a little more about what branch your commit is on? And is this > a commit created by `git-webkit upload`, or is it one you created with `git > commit`. This behavior is currently tied to the creation of a PR branch, > `git-webkit branch` while main is checked out is the more narrow example. So I'm in main branch. Then I do "git checkout -b my-new-branch", and I work on that one on a patch. And then I use "git commit" to create a commit for my patch. When I do "git-webkit upload" I don't get asked about the bug thing. In the past with "webkit-patch upload -g HEAD" I did get asked for the bug thing. Even without committing changes I could do the same with "webkit-patch". I've just tried a similar workflow from main: git checkout -b my-new-branch Tools/Scripts/git-webkit commit Tools/Scripts/git-webkit upload Still no prompt to create a new bug. It's true that if I just do "Tools/Scripts/git-webkit branch" I get the prompt for entering the issue URL or tile of the new issue. 1870855 8 jbedard 2022-05-20 16:02:30 -0700 (In reply to Manuel Rego Casasnovas from comment #7) > (In reply to Jonathan Bedard from comment #6) > > Can you share a little more about what branch your commit is on? And is this > > a commit created by `git-webkit upload`, or is it one you created with `git > > commit`. This behavior is currently tied to the creation of a PR branch, > > `git-webkit branch` while main is checked out is the more narrow example. > > So I'm in main branch. > > Then I do "git checkout -b my-new-branch", and I work on that one on a > patch. And then I use "git commit" to create a commit for my patch. > > When I do "git-webkit upload" I don't get asked about the bug thing. > In the past with "webkit-patch upload -g HEAD" I did get asked for the bug > thing. Even without committing changes I could do the same with > "webkit-patch". > > I've just tried a similar workflow from main: > git checkout -b my-new-branch > Tools/Scripts/git-webkit commit > Tools/Scripts/git-webkit upload > > Still no prompt to create a new bug. > > It's true that if I just do "Tools/Scripts/git-webkit branch" I get the > prompt for entering the issue URL or tile of the new issue. It's the `-b my-new-branch` part that is preventing the prompt....the assumption is that by providing a branch name instead of a bug title, you don't want us to create a new bug. This might be a bad assumption, which is why I'm wondering if `git-webkit commit` should assume you want to create a bug if you don't pass a bug ID and aren't amending a commit. 1871435 9 rego 2022-05-24 00:00:38 -0700 (In reply to Jonathan Bedard from comment #8) > It's the `-b my-new-branch` part that is preventing the prompt....the > assumption is that by providing a branch name instead of a bug title, you > don't want us to create a new bug. This might be a bad assumption, which is > why I'm wondering if `git-webkit commit` should assume you want to create a > bug if you don't pass a bug ID and aren't amending a commit. Ok, thanks. Now I understand how it works. Just to explain my workflow, in case you find it useful. I do local branches and local commits all the time. Most of them are basically experiments here and there or whatever. Things mostly for myself. Once I have a branch that has something useful, then I run the "git-webkit upload" command. At that point is when I'd like to get asked about creating a new bug or not. Even at that point I'd be happy to provide a bug number, if it already exists, to pre-fill some of the commit message with it. Probably other people workflow differs. But this workflow was working nicely with "webkit-patch upload". And it also works in Chromium, where the "magic" only happens when you do "git cl upload" (the local branches and commits are totally independent of the rest of stuff).