feat(treescript): Bug 2015581 Add github create branch action #1341
Conversation
hneiva
force-pushed
the
hneiva/ios-merge-automation
branch
2 times, most recently
from
9b3ef08 to
302b928
Compare
Eijebong
left a comment
Eijebong
left a comment
There was a problem hiding this comment.
A bunch of minor things. Also I would've preferred if this had been split in 2 commits (I'm not even sure submitting the whole thing was intentional), the version bump in xcconfig and the branch creation since they're completely unrelated in code. Neither the commit nor the bug even mentions that version bump semantic change.
| repo = (await self._client.execute(str_info_query))["repository"] | ||
|
|
||
| if repo.get("object") is None: | ||
| raise UnknownBranchError(f"branch '{source_branch}' not found in repo!") |
There was a problem hiding this comment.
I wonder if we should also check if the branch already exists and bail early? I'm seeing the "branch creation passed but version bump got 500s" already...
Although there's an argument for saying that if the branch already exists it could've been created by mistake by a human and shouldn't be used and thus failing here is the right thing to do...
hneiva
force-pushed
the
hneiva/ios-merge-automation
branch
from
302b928 to
65e421a
Compare
hneiva
force-pushed
the
hneiva/ios-merge-automation
branch
from
65e421a to
0340d98
Compare
2 participants
No description provided.