Outdated item on the "before merge" checklist? #5842
|
We have an item on the PR template checklist: Given that @wdanilo as you have power over GH templates. |
Answered by
wdanilo
Mar 8, 2023
Replies: 1 comment 2 replies
|
@farmaazon |
2 replies
Answer selected by
farmaazon
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
@farmaazon
./run ide watchuses Electron that is not packaged, it hacks into Electron and runs it with our content. There MIGHT be cases where the./run ide buildwill fail while./run ide watchwill not. However, I do not see why the opposite could happen. I mean, if./run ide buildsucceeds and the electron app works,./run ide watchshould as well. So I'd remove the later. What about that?