Slackbot
03/09/2023, 5:09 PMAryaman
03/09/2023, 5:11 PMJordan Gensler
03/09/2023, 5:14 PMBen Jaffe
03/09/2023, 5:43 PMPrasanth Somasundar
03/10/2023, 8:59 AMAryaman
03/10/2023, 12:50 PMBen Jaffe
03/10/2023, 3:30 PMPrasanth Somasundar
03/10/2023, 7:04 PMPrasanth Somasundar
03/10/2023, 7:04 PMAryaman
03/10/2023, 7:35 PMIs that because those users don’t want to force their teammates to use graphite?Yes exactly. cc: @merrill & @Xiulung Choy on this feedback
Ben Jaffe
03/10/2023, 7:38 PMBilal Quadri
03/10/2023, 8:02 PMBilal Quadri
03/11/2023, 9:30 AM// ==UserScript==
// @name Graphite Github Button
// @namespace <https://zoya.finance>
// @version 1.0
// @description Create a nav link to open Github PR from Graphite
// @author Bilal Quadri
// @match <https://app.graphite.dev/*>
// @icon <https://www.google.com/s2/favicons?sz=64&domain=graphite.dev>
// @grant none
// ==/UserScript==
(function() {
'use strict';
const navBarSelector = '.pull-request-title-bar'
const githubButtonId = 'tampermonkey-github-button'
const displayLink = () => {
if (document.getElementById(githubButtonId)) {
return
}
const navBar = document.querySelector(navBarSelector)
if (!navBar) {
return
}
const [, owner, repo, prNumber] = window.location.href.match(/\/github\/pr\/(.+)\/(.+)\/([0-9]+).*/)
const githubLink = `<https://github.com/${owner}/${repo}/pull/${prNumber}>`
const githubButton = document.createElement('div')
githubButton.innerHTML = `
<a class="buttons__button" id="${githubButtonId}" href="${githubLink}">
<svg xmlns="<http://www.w3.org/2000/svg>" fill="#ffffff" width="24" height="24" viewBox="0 0 24 24">
<path d="M12 0c-6.626 0-12 5.373-12 12 0 5.302 3.438 9.8 8.207 11.387.599.111.793-.261.793-.577v-2.234c-3.338.726-4.033-1.416-4.033-1.416-.546-1.387-1.333-1.756-1.333-1.756-1.089-.745.083-.729.083-.729 1.205.084 1.839 1.237 1.839 1.237 1.07 1.834 2.807 1.304 3.492.997.107-.775.418-1.305.762-1.604-2.665-.305-5.467-1.334-5.467-5.931 0-1.311.469-2.381 1.236-3.221-.124-.303-.535-1.524.117-3.176 0 0 1.008-.322 3.301 1.23.957-.266 1.983-.399 3.003-.404 1.02.005 2.047.138 3.006.404 2.291-1.552 3.297-1.23 3.297-1.23.653 1.653.242 2.874.118 3.176.77.84 1.235 1.911 1.235 3.221 0 4.609-2.807 5.624-5.479 5.921.43.372.823 1.102.823 2.222v3.293c0 .319.192.694.801.576 4.765-1.589 8.199-6.086 8.199-11.386 0-6.627-5.373-12-12-12z"/>
</svg>
</a>
`
navBar.insertBefore(githubButton, navBar.lastChild)
}
// Run on interval in case React re-rendering removes it for some reason.
// Doesn't seem to be an issue right now, but no need to risk it.
setInterval(displayLink, 1000)
})();
Ben Jaffe
03/13/2023, 12:07 AMGabrielle Delforge
03/13/2023, 1:52 PMJaisal Friedman
03/15/2023, 8:39 PMAryaman
03/28/2023, 2:37 PMJordan Gensler
03/28/2023, 4:00 PMBen Jaffe
03/28/2023, 4:28 PMAryaman
03/28/2023, 4:39 PMMatthew Keller
03/29/2023, 10:10 PMmerrill
03/30/2023, 1:02 AMcodeowners
info) we’ll prioritize adding the missing info to Graphite quickly (as we’re doing right now). For more complicated interfaces such as detailed CI logs and raw file views, we’ll link out to the relevant page in GitHub in a new tab.
Again, appreciate the push on this - we know this probably isn’t the answer you want to hear, but hopefully the progress the team here has made on the dashboard over the past few months gives you some confidence that we’ll keep working hard until this doesn’t feel like a pain point. 🙏Jordan Gensler
03/30/2023, 1:31 AMBen Jaffe
03/30/2023, 1:32 AMMatthew Keller
03/30/2023, 3:50 PMmerrill
03/30/2023, 8:53 PMcodeowners
): @Brendan Ngo is actively working on ingesting & displaying codeowners
on Graphite. We’ll also take a look at adding a typeahead for GH Issues with #
, as well as searching for repos, files etc.
• Correctness/reliability & performance: @Brendan Ngo, @Ziyao Wei, & @Nicholas Yan are working on a multi-month infra project that will address the known issues around Graphite misreporting PR/mergeability statuses. Their work should also yield significant improvements in PR page load times (in addition to major improvements we’ve made here in the past couple months).
• Familiarity & new user experience: We’re planning changes to help new users better configure and understand how to use the Graphite PR page more intuitively.
A couple more changes in-flight to help with GH interop:
• @Jacob Gold is updating our Chrome extension to be less aggressive - it will soon just add a button to the PR page in GH to let you navigate back to Graphite vs. the (buggy) auto-redirect behavior.
• We’ll prioritize fixing the redirect on Graphite links to GH for logged out users.
@Ben Jaffe I like the crate training metaphor - could be an interesting hypothesis for us to test with a cohort of new users!
@Jordan Gensler when you say “90% of my coworkers will not use Graphite” - is there a particular reason why they wouldn’t ever give it a try?
@Matthew Keller heard re: not getting value out of the review experience today - it’s on us to keep pushing and iterating on the review experience until it feels compelling, and that’s what we’re going to do. At a high level we’re not here to build a piecemeal assortment of features and plugins - our goal is to build a highly integrated workflow tool that helps our users create, review, and merge code changes more efficiently. That doesn’t meant that we expect everyone to use every part of the experience we build, but our priority is to help more users and teams adopt Graphite end-to-end and provide the best possible experience for those who do.Jordan Gensler
03/31/2023, 3:56 AMBilal Quadri
03/31/2023, 10:15 AM@@ -211,7 +211,7 @@ export class DocumentEmailStateMachine extends Construct {
g. Other things too that I can’t remember off the top of my head
3. There are cases where reliability issues require me to jump to Github for a source of truth. PRs not appearing as part of a stack when they should, merge job failures, etc. This is pretty rare, but I like having the escape hatch for peace of mind. Lately, Github itself has had many issues, and it’s helpful to be able to pop over and confirm that it is indeed Github at the root cause of the issue.
4. It’s a personal preference, but I like the syntax highlighting and background color of the theme on Github Dark more than Graphite. Sometimes, when a PR is hard for me to make sense of, I like to open it in Github to get more comfortable readability. The actual diff highlighting is better on Graphite though. Then again, the lack of contexts on diffs still has overall easier reading on Github.
5. Would love a way to be able to flip the sidebar to be on the left side instead. Github has it that way, and it’s just more comfortable for me since literally every other place I read code also does it that way (Github, VSCode, Neovim CHADTree, etc).
6. Sometimes I want to go to Github to search/read other code in the repo that may be affected by the change but aren’t in the diff. Can’t do that in Graphite so I need to be able to popover to Github quickly for that flow.
7. It was already mentioned and you already have plans for it, but mobile is pretty broken. Clicking on a graphite slack notification from my phone is frustrating. In the short term, I can probably solve the pain by just logging out of Graphite on mobile so it has the go to Github button again.Leslie Lam
04/03/2023, 8:28 PMBen Jaffe
04/04/2023, 2:28 PMBilal Quadri
04/04/2023, 8:56 PMBen Jaffe
06/30/2023, 8:48 PMmerrill
07/01/2023, 3:34 PMBen Jaffe
07/02/2023, 2:37 AM