Kevin Goslar

Results 334 comments of Kevin Goslar
trafficstars

Suggestion for a fix: Simply ignore loops that contain a `continue` statement. Avoiding false positives feels more important than having a few false negatives. ChatGPT can prototype the code to...

Great ideas! I agree with you that `GetAPIToken` should be called when constructing the config data structures. The current behavior is buggy. I think it's okay if `git town config`...

I have implemented a spike of the remaining feature for this ticket: showing where config values come from. After exploring it, I don't think the value justifies the added complexity....

Some things that Git Town could do here: - display a proper error message - provide a config setting for `.git-branches.toml` to disable ship for this repo since it uses...

Good point. I forgot that GitHub's merge queue really only replaces the "merge" button and doesn't do the testing. `git ship` should detect the merge queue and merge the PR...

Your questions are legit. I'm planning a series of screencasts that explain this better in the coming weeks. > create PR on GH, wait for approval (should I use git...

@stephenwade You are right that the actual commits in the branches don't get updated. You can update them by running `git town sync --all`. I meant that GitHub updates the...

Beautiful edge case, thanks for reporting this! 🙏 This will get fixed in the way you describe. Just out of curiosity, do you have a real-life scenario where somebody doesn't...

Thanks for bringing this up! The documentation for this definitely needs improvement, so here is a more detailed explanation of why this works the way it does right now. Merge...