Y
HN Search
Hacker News Search
new
|
comments
|
top
|
jobs
sameenkarim
searching Neon…
1.
▲
2.
▲
3.
▲
4.
▲
5.
▲
6.
▲
7 ms
·
1.
▲
by
sameenkarim
2mo ago
Yes it will be coming! The reason it's taking a bit longer is because of the automated rebase that happens after you merge part of a stack. There are some legitimate security concerns because of this so multi-fork stacks (a stack which
2.
▲
by
sameenkarim
2mo ago
The draft one can't be merged, but the one below it isn't affected and can be merged as long as it's green. The dependency goes bottom-up.
3.
▲
by
sameenkarim
2mo ago
Thanks, definitely sounds like a bug. Will look into this.
4.
▲
by
sameenkarim
2mo ago
Yes that was one of our top priorities. For one, there's a fully public API for all stack operations. So if you don't want to use the `gh stack` CLI, you can build your own: https://docs.github.com/en/rest
5.
▲
by
sameenkarim
2mo ago
We've been using the pancake emoji internally so we thought it was a fun easter egg. It'll only be up for a few hours and then will go back to the regular icons :)
6.
▲
by
sameenkarim
2mo ago
I hear you. We had to start with something a bit more minimal, but we are working on a much broader revamp for the PR UI. Part of that will include a persistent view of the stack so you always know you're working with a stack and easil
7.
▲
by
sameenkarim
2mo ago
We're rolling out a series of bug fixes for the issues with squash merging. There's an internal system we have called CPRMC (Create Pull Request Merge Commit) that is used to evaluate whether a PR is "ready" to merge. Th
8.
▲
by
sameenkarim
2mo ago
Hey from the GitHub Stacked PRs team! Excited to release this more broadly so anyone can start stacking: https://gh.io/stacks Would love to hear any feedback, especially with the UI and CLI. We've got a lot more update
9.
▲
by
sameenkarim
5mo ago
When you're using the UI to open a PR, if you set the base to a branch that has an open PR there'll be an "Add to Stack" option: https://github.github.com/gh-stack/guides/ui/#step-2-create
10.
▲
by
sameenkarim
5mo ago
Yes, we handle this both in the CLI and server using git rebase --onto git rebase --onto <new_commit_sha_generated_by_squash> <original_commit_sha_from_tip_of_merged_branch> <branch_name> So for ex in this scenario:
11.
▲
by
sameenkarim
5mo ago
We're shipping a skill file with the CLI: https://skills.sh/github/gh-stack/gh-stack Everyone will have their own way of structuring stacks, but I've found it great for the agent to plan a stack structur
12.
▲
by
sameenkarim
5mo ago
Stacked PRs can be created via the UI, API, or CLI. You can also run a combination of these. For ex, use another tool like jj to develop locally, push up the branches, and use the gh CLI to batch create a stack of n PRs, without touching lo
13.
▲
by
sameenkarim
5mo ago
> So, if I have three patches in the stack, and I want to merge the bottom two, I'd merge one, wait for tests to run on the other, merge the second vs. merge just those two in one step As we have it designed currently, you would hav
14.
▲
by
sameenkarim
5mo ago
Yup, there will be an API for stacks, just like there is one for regular PRs.
15.
▲
by
sameenkarim
5mo ago
The CLI is not required and you can push up your bookmarks as branches and open stacked PRs via the UI. You can also use the gh CLI to just create the stacked PRs on github.com (essentially an API wrapper), without using it to manage local
16.
▲
by
sameenkarim
5mo ago
The CLI is completely optional, you can create stacked PRs purely via the UI. Also the rationale for having a chain of branches pointing to each other was so the diff in a PR shows just the relevant changes from the specific branch, not the
17.
▲
by
sameenkarim
5mo ago
Yeah features need to be released as GA (general availability) before they can be included in GHES. I don't have a definitive timeline, but it will likely be end of this year or early next.
18.
▲
by
sameenkarim
2y ago
Wow, that's a lot of events! Hope this helps with keeping track of it all now.
19.
▲
by
sameenkarim
2y ago
Glad to hear I'm not alone in the frustration! Thanks and let me know how it goes.
20.
▲
Show HN: Auto-document your analytics setup (npx package)
(github.com)
17 points
by
sameenkarim
2y ago
|
5 comments
21.
▲
Show HN: I made an app to set my status emoji in Slack
(slacklunchstatus.com)
3 points
by
sameenkarim
5y ago
|
0 comments