This message was deleted.
# report-bugs
s
This message was deleted.
1
a
@Adam Mustafa, which PR count is off? The number of PRs queued to merge, or number merged in the past 7 days?
a
Ah, I see. I figured it was the total (which would have included the 11)
my mistake
a
It will update once they are merged!
So next time you should see it go up to 14.
a
I have problems in general merging my stack. It always fails on the second job because we use squash and merge.
Do you have any suggestions for that?
a
cc @Jacob Gold who is working on making the merging experience for stacks more reliable.
j
do you have an example of a PR that failed to merge that you expected to succeed? we are working on adding better error messaging, but I can look up why specific jobs failed for now.
a
Sure, I'm trying right now. Those 11 PRs, the second failed.
How can I help diagnose. I still have 10 to merge
j
Interesting error! I haven't seen this one before —
Copy code
Command failed: git push -f origin am/replace-transactions-w-journals\nremote: error: GH007: Your push would publish a private email address.        \nremote: You can make your email public or disable this protection by visiting:        \nremote: <http://github.com/settings/emails>        \nTo <https://github.com/FinTronners/FTron>\n ! [remote rejected] am/replace-transactions-w-journals -> am/replace-transactions-w-journals (push declined due to email privacy restrictions)\nerror: failed to push some refs to '<https://github.com/FinTronners/FTron>'\n"
But yeah, we will be surfacing errors like this more clearly in the coming weeks.
1
a
If it helps, after every merge I get this notification from Github:
a
That is because of GitHub not knowing about stacks first class. You should be able to fix the error that Jacob linked above by unchecking attached setting in
<http://github.com/settings/emails|github.com/settings/emails>
.
@Jacob Gold, i wonder if we can support users who have this setting checked?
j
Re the merge conflicts message — those merge conflicts are resolved by a rebase, which the graphite merge performs automatically for you. So that notification is nothing to worry about.
a
That's what I figured. I assumed it was due to the squash and merge behavior
👍 1
Also getting this message now: Hope I'm not being spammy. I just love the app and trying to help.
j
Not at all! I'm actively working on this, so it's quite helpful