{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":645497880,"defaultBranch":"main","name":"sapling","ownerLogin":"alex-statsig","currentUserCanPush":false,"isFork":true,"isEmpty":false,"createdAt":"2023-05-25T19:42:04.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/77301670?v=4","public":true,"private":false,"isOrgOwned":false},"refInfo":{"name":"","listCacheKey":"v0:1721883434.0","currentOid":""},"activityList":{"items":[{"before":"abf06f135193984a17bf46be8965733ed777db75","after":"1f96720578c539ded5cfb43f5184688a4d09f983","ref":"refs/heads/sapling-pr-archive-alex-statsig","pushedAt":"2024-07-25T04:57:19.000Z","pushType":"push","commitsCount":3196,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"merge commit for archive created by Sapling","shortMessageHtmlLink":"merge commit for archive created by Sapling"}},{"before":null,"after":"986844b14436b785a954024d0a88d5e4a1c1944d","ref":"refs/heads/pr926","pushedAt":"2024-07-25T04:57:14.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Fix handling of GHE without merge queue support\n\nSummary:\nhttps://github.com/facebook/sapling/issues/830 and https://github.com/facebook/sapling/issues/906 reported sapling not working for old Github Enterprise versions since their graphql schema doesnt include merge queue.\n\nThis adds a query to detect if merge queue is supported in the graphql version. Then, it will issue a different \"YourPullRequestsQuery\" depending on if merge queue is supported.\n\nI considered a few other approaches:\n- Just detect the error message and retry without merge queue field - a little gross to extract from the error\n- Use \"@include\" instead of an entirely separate query - didn't seem to count as a valid query still via graphiql at least (unsure if the query client here would strip out the unused field though)\n\nTest Plan:\nHaven't had a chance to test yet, will try and validate that both versions work and that the detection logic works.","shortMessageHtmlLink":"Fix handling of GHE without merge queue support"}},{"before":"2fe60ca9f8d1106987d1b953cd80520e39754ae0","after":"5f7a862b24e287daedeea0cca8617da639f0a013","ref":"refs/heads/main","pushedAt":"2024-07-25T04:49:57.000Z","pushType":"push","commitsCount":3194,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"shelve: exit unfinished merge state after interactive shelve\n\nSummary:\nAccording to the QA page [1], in some situations, sapling ends into the\nconfusing `The repository is in an unfinished *merge* state`. Jun introduced a\n`sl continue` command before and we print a help message telling user to run\nthe command.\n\nIntead of asking users to run `sl continue`, this diff just run the state\ncleanup logic automatically at end of `shelve` command.\n\nWe also need to dive deep into the `goto` command to understand why it\nresults into this unfinished merge state. But for now, let's use this workaround to\nimprove user experience.\n\nReviewed By: muirdm\n\nDifferential Revision: D60157205\n\nfbshipit-source-id: 61f35b7e6a6f47bf84f5f3718255bae25cf00ccd","shortMessageHtmlLink":"shelve: exit unfinished merge state after interactive shelve"}},{"before":"e8082f272b64c24085ae5ccc96ccfe42e165bc26","after":"abf06f135193984a17bf46be8965733ed777db75","ref":"refs/heads/sapling-pr-archive-alex-statsig","pushedAt":"2024-02-24T08:37:21.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"merge commit for archive created by Sapling","shortMessageHtmlLink":"merge commit for archive created by Sapling"}},{"before":"e2722899090a07bc321b70cb750c3ce4e9102f95","after":"8ea57fdcd766ff47121b177819e2986afcf0d153","ref":"refs/heads/pr816","pushedAt":"2024-02-24T08:37:16.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Add \"open in code review\" context menu action\n\nA common feature from other SCM extensions is a context-menu item to copy a link / open the selection/file in the remote SCM provider (ex. open in Github). This adds support for that in the editor / file explorer.\n\nCouple of things I wasn't totally confident about that I'd like thoughts on:\n- I opted to just add this to context menu in vscode since that's where most other extensions do this, but I'm guessing this might be desired in ISL too, which might changed some implementation details\n- Open to other titles for the commands, but afaict the message can't be dynamic (i.e. use code review provider's name), so I settled on \"Open in Code Review\"\n- I went with the simple approach of copying a link to the file on main using the same line numbers. This won't necessarily work if the file is dirty or modified in remote main. This could be mitigated by using the most recent public commit instead of main, and perhaps some magic with blame, but seemed more complex\n- Not sure the best way to grab the remote branch's name; just hard-coded \"main\" in my testing, but it could be \"master\" or something else as well. Is this easily retrievable from sapling?\n- I started setting a context for \"sapling:hasRepo\" so the option would only show if the editor has some folder with a sapling repo. I could be smarter and check if the active file is in a repo, but this seemed effective enough and simpler (scmProvider used in other commands is only available with the SCM panel)","shortMessageHtmlLink":"Add \"open in code review\" context menu action"}},{"before":"98d26411f45179e6fea15907712efe82f8345059","after":"e8082f272b64c24085ae5ccc96ccfe42e165bc26","ref":"refs/heads/sapling-pr-archive-alex-statsig","pushedAt":"2024-02-24T08:34:22.000Z","pushType":"push","commitsCount":769,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"merge commit for archive created by Sapling","shortMessageHtmlLink":"merge commit for archive created by Sapling"}},{"before":"2006eef7853667e945d007d51182a51996753a28","after":"e2722899090a07bc321b70cb750c3ce4e9102f95","ref":"refs/heads/pr816","pushedAt":"2024-02-24T08:34:19.000Z","pushType":"force_push","commitsCount":0,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Add \"open in code review\" context menu action\n\nA common feature from other SCM extensions is a context-menu item to copy a link / open the selection/file in the remote SCM provider (ex. open in Github). This adds support for that in the editor / file explorer.\n\nCouple of things I wasn't totally confident about that I'd like thoughts on:\n- I opted to just add this to context menu in vscode since that's where most other extensions do this, but I'm guessing this might be desired in ISL too, which might changed some implementation details\n- Open to other titles for the commands, but afaict the message can't be dynamic (i.e. use code review provider's name), so I settled on \"Open in Code Review\"\n- I went with the simple approach of copying a link to the file on main using the same line numbers. This won't necessarily work if the file is dirty or modified in remote main. This could be mitigated by using the most recent public commit instead of main, and perhaps some magic with blame, but seemed more complex\n- Not sure the best way to grab the remote branch's name; just hard-coded \"main\" in my testing, but it could be \"master\" or something else as well. Is this easily retrievable from sapling?\n- I started setting a context for \"sapling:hasRepo\" so the option would only show if the editor has some folder with a sapling repo. I could be smarter and check if the active file is in a repo, but this seemed effective enough and simpler (scmProvider used in other commands is only available with the SCM panel)","shortMessageHtmlLink":"Add \"open in code review\" context menu action"}},{"before":"f1444664e7bb756549fa00d4053d39f242c42ce0","after":"2fe60ca9f8d1106987d1b953cd80520e39754ae0","ref":"refs/heads/main","pushedAt":"2024-02-24T07:38:33.000Z","pushType":"push","commitsCount":767,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Updating submodules\n\nSummary:\nGitHub commits:\n\nhttps://github.com/facebook/cachelib/commit/34a165993083d71b442c89b9ab86e0c34682d0c8\nhttps://github.com/facebook/fatal/commit/fa438c98f87e9693f7721bb3196db9edc537acf2\nhttps://github.com/facebook/fb303/commit/f2d9ebb68b6a28addeab2cc75676d145fa440595\nhttps://github.com/facebook/fbthrift/commit/7352c9ffa672d910f8a32190bdddac18043ca2db\nhttps://github.com/facebook/folly/commit/ec680d269386c81af5e2aae0ae543432db13827b\nhttps://github.com/facebook/litho/commit/332029fe818238e579aadd845a1642315555c859\nhttps://github.com/facebook/mcrouter/commit/23892d22d3717650a4cb8aa80acc5c9cd1cd7ff0\nhttps://github.com/facebook/mvfst/commit/05c2df2e867a9a5bc23b9020e28d447cca039c88\nhttps://github.com/facebook/ocamlrep/commit/707aa9b1cda0ac4bd2fcf167f2c43009aa40dce3\nhttps://github.com/facebook/proxygen/commit/9f939745c60d64040a6849fef338c5073a843515\nhttps://github.com/facebook/rocksdb/commit/2940acac0088384b9fd146ce2a2ba5e2a298d5d5\nhttps://github.com/facebook/wangle/commit/dd4425b49d8ee4cbc3cf022f56721967df329ed6\nhttps://github.com/facebook/watchman/commit/05e6c908d14681e928eaf69d9d2dd5aa288cc1d8\nhttps://github.com/facebookexperimental/edencommon/commit/11c0b6b8c2d996f927325c54d657f8ebd6df572e\nhttps://github.com/facebookexperimental/rust-shed/commit/9deb5e943938cd32e9a2a0abadb2cfae9fb52584\nhttps://github.com/facebookincubator/fizz/commit/a1a39764cd05c7b8c46bb37f5e305c9b81919995\nhttps://github.com/facebookincubator/katran/commit/b2bb68ba7d131e82ae8741acdc6d0657830505ad\nhttps://github.com/facebookincubator/velox/commit/3e4a26a05ae96c56af0eec0b274d2dd8633733f0\nhttps://github.com/pytorch/fbgemm/commit/dc209666f0ee78d404ed20b269fe17f7934074c2\n\nReviewed By: jailby\n\nfbshipit-source-id: a189d0888ca0739de17c4fd236ae515ebab7866d","shortMessageHtmlLink":"Updating submodules"}},{"before":"96cbe5cc41e415d29aaacd6c0e4583af3bff635c","after":"69469792b29549da43407f27ce90d0aef83caaf1","ref":"refs/heads/pr827","pushedAt":"2024-02-04T16:28:09.000Z","pushType":"push","commitsCount":209,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Merge branch 'main' into pr827","shortMessageHtmlLink":"Merge branch 'main' into pr827"}},{"before":"b96bb3629e5d7178b556863fda23f3392dcef5ed","after":"94c8c3ac3a133d9e8dbb14cec922b0fb14850c76","ref":"refs/heads/pr717","pushedAt":"2024-02-04T16:26:30.000Z","pushType":"push","commitsCount":3317,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Merge branch 'main' into pr717","shortMessageHtmlLink":"Merge branch 'main' into pr717"}},{"before":"21ad848f6e41176867595e44ef0d6e8b5a81689a","after":"98d26411f45179e6fea15907712efe82f8345059","ref":"refs/heads/sapling-pr-archive-alex-statsig","pushedAt":"2024-01-26T19:33:31.000Z","pushType":"push","commitsCount":205,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Merge 96cbe5cc41e415d29aaacd6c0e4583af3bff635c into sapling-pr-archive-alex-statsig","shortMessageHtmlLink":"Merge <a class=\"commit-link\" data-hovercard-type=\"commit\" data-hovercard-url=\"https://github.com/alex-statsig/sapling/commit/96cbe5cc41e415d29aaacd6c0e4583af3bff635c/hovercard\" href=\"https://github.com/alex-statsig/sapling/commit/96cbe5cc41e415d29aaacd6c0e4583af3bff635c\"><tt>96cbe5c</tt></a> into sapling-pr-archive-alex-statsig"}},{"before":null,"after":"96cbe5cc41e415d29aaacd6c0e4583af3bff635c","ref":"refs/heads/pr827","pushedAt":"2024-01-26T19:33:29.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Don't auto-close drawer when window isnt loaded\n\nThis was triggering frequently for me, auto-closing my ISL after I closed the sidebar and reopened it later.\n\nThis case seems irrelevant and likely indicative that the frame just isnt loaded yet, so just skipping the autoclose.","shortMessageHtmlLink":"Don't auto-close drawer when window isnt loaded"}},{"before":"25d6b85bb2a45f583cd1a689a60f3288b540e1ea","after":"f1444664e7bb756549fa00d4053d39f242c42ce0","ref":"refs/heads/main","pushedAt":"2024-01-25T00:10:43.000Z","pushType":"push","commitsCount":203,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"removed deleted tunables\n\nSummary: removed deleted tunables\n\nReviewed By: YousefSalama\n\nDifferential Revision: D53050767\n\nfbshipit-source-id: 54ac3101055fb808c99fc09e21c5e9b3d43e6f6a","shortMessageHtmlLink":"removed deleted tunables"}},{"before":"1882bd501374044e4f394b42c18352ba0ee8b94f","after":"ffb778fb8c41ffcb569faff23b7076dec45308d9","ref":"refs/heads/pr718","pushedAt":"2024-01-24T18:37:53.000Z","pushType":"push","commitsCount":3102,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Merge branch 'main' into pr718","shortMessageHtmlLink":"Merge branch 'main' into pr718"}},{"before":"e6e1c8d962701b1b9bc5ebab6e998970bbcf2bd4","after":"21ad848f6e41176867595e44ef0d6e8b5a81689a","ref":"refs/heads/sapling-pr-archive-alex-statsig","pushedAt":"2024-01-21T01:07:47.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Merge be417bb7ee9e81b9921eb577d44f14d88ac18ffa into sapling-pr-archive-alex-statsig","shortMessageHtmlLink":"Merge <a class=\"commit-link\" data-hovercard-type=\"commit\" data-hovercard-url=\"https://github.com/alex-statsig/sapling/commit/be417bb7ee9e81b9921eb577d44f14d88ac18ffa/hovercard\" href=\"https://github.com/alex-statsig/sapling/commit/be417bb7ee9e81b9921eb577d44f14d88ac18ffa\"><tt>be417bb</tt></a> into sapling-pr-archive-alex-statsig"}},{"before":null,"after":"be417bb7ee9e81b9921eb577d44f14d88ac18ffa","ref":"refs/heads/pr822","pushedAt":"2024-01-21T01:07:44.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Don't include commit title in PR body\n\nSeems redundant to include the commit title in both the PR body and title, so this strips out the title.\n\nHad some trouble building to test this out (ran into `error[E0658]: use of unstable library feature 'stdsimd'`)","shortMessageHtmlLink":"Don't include commit title in PR body"}},{"before":"d36177c662016c1841875b628799aad3f7a7c8df","after":"e6e1c8d962701b1b9bc5ebab6e998970bbcf2bd4","ref":"refs/heads/sapling-pr-archive-alex-statsig","pushedAt":"2024-01-21T00:00:02.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Merge 19a57a5b3179dca88f86409f1c706a20b229fdb7 into sapling-pr-archive-alex-statsig","shortMessageHtmlLink":"Merge <a class=\"commit-link\" data-hovercard-type=\"commit\" data-hovercard-url=\"https://github.com/alex-statsig/sapling/commit/19a57a5b3179dca88f86409f1c706a20b229fdb7/hovercard\" href=\"https://github.com/alex-statsig/sapling/commit/19a57a5b3179dca88f86409f1c706a20b229fdb7\"><tt>19a57a5</tt></a> into sapling-pr-archive-alex-statsig"}},{"before":null,"after":"19a57a5b3179dca88f86409f1c706a20b229fdb7","ref":"refs/heads/pr821","pushedAt":"2024-01-20T23:59:58.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Fix commit message template\n\nFixing an issue where the commit template from `sl debugcommitmessage` would be overridden due to `Internal.getCustomDefaultCommitTemplate`. This is because the check `customTemplate?.trim() !== ''` would return `true` for undefined, leading to the template being overwritten with `undefined`.\n\nAfter this fix, a template correctly loads into ISL with \"Summary\" and \"Test Plan\" by default.\n\nAlso updated the \"ignore line\" regex to include the trailing linebreak (optionally), to prevent a bunch of blank lines in templates.\n\n## Test Plan\n\nConfirmed after this fix, the default template of:\n```\n<Replace this line with a title. Use 1 line only, 67 chars or less>\n\nSummary:\n\nTest Plan:\n\n\nSL: Enter commit message.  Lines beginning with 'SL:' are removed.\nSL: Leave message empty to abort commit.\nSL: --\nSL: user: Alex Coleman <alex@statsig.com>\nSL: changed addons/isl-server/src/Repository.ts        |  5 +++--\nSL: changed addons/isl-server/src/ServerToClientAPI.ts |  2 +-\nSL:  2 files changed, 4 insertions(+), 3 deletions(-)\n```\n\nwas now transformed to:\n```\n\nSummary:\n\nTest Plan:\n\n\n```\n\nAnd then correctly appeared in ISL:","shortMessageHtmlLink":"Fix commit message template"}},{"before":"238716b80933f70d75209896bb7ac939efe38eb8","after":"d36177c662016c1841875b628799aad3f7a7c8df","ref":"refs/heads/sapling-pr-archive-alex-statsig","pushedAt":"2024-01-20T23:59:26.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Merge daea6a4fdb04168fc06ca796a976bd05f7ac626f into sapling-pr-archive-alex-statsig","shortMessageHtmlLink":"Merge <a class=\"commit-link\" data-hovercard-type=\"commit\" data-hovercard-url=\"https://github.com/alex-statsig/sapling/commit/daea6a4fdb04168fc06ca796a976bd05f7ac626f/hovercard\" href=\"https://github.com/alex-statsig/sapling/commit/daea6a4fdb04168fc06ca796a976bd05f7ac626f\"><tt>daea6a4</tt></a> into sapling-pr-archive-alex-statsig"}},{"before":null,"after":"daea6a4fdb04168fc06ca796a976bd05f7ac626f","ref":"refs/heads/pr820","pushedAt":"2024-01-20T23:59:22.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Trim commit description from SSL\n\n\nCurrently, when a commit is created with a title/description, the PR/commit description is:\n```\n{title}\n\n{desc}\n```\n\nWhen the description was parsed out previously, the first line (and trailing linebreak) were stripped, but the line in between title/desc was left around. This meant every time you amended a commit/PR, another linebreak got added to the top.\n\nI figure its probably fine to just trim whitespace from the description in general, but could update this to only trim leading whitespace (cant really just change the slice() index to ` + 2` because the commit may come from another source than sapling, and may not contain this extra linebreak)","shortMessageHtmlLink":"Trim commit description from SSL"}},{"before":"9939f61a58df9b854298eee0a1ce93af3e8c0072","after":"238716b80933f70d75209896bb7ac939efe38eb8","ref":"refs/heads/sapling-pr-archive-alex-statsig","pushedAt":"2024-01-18T17:39:52.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Merge 0a64bc45b97ff0a1bcf43c9707baac1eddc22aaf into sapling-pr-archive-alex-statsig","shortMessageHtmlLink":"Merge <a class=\"commit-link\" data-hovercard-type=\"commit\" data-hovercard-url=\"https://github.com/alex-statsig/sapling/commit/0a64bc45b97ff0a1bcf43c9707baac1eddc22aaf/hovercard\" href=\"https://github.com/alex-statsig/sapling/commit/0a64bc45b97ff0a1bcf43c9707baac1eddc22aaf\"><tt>0a64bc4</tt></a> into sapling-pr-archive-alex-statsig"}},{"before":null,"after":"0a64bc45b97ff0a1bcf43c9707baac1eddc22aaf","ref":"refs/heads/pr817","pushedAt":"2024-01-18T17:39:48.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Default to showing blame\n\nBlame feature has been out for a while and seems functional generally. I often get questions about how to enable this.\n\nSeems like it should be good to just enable by default, but if that conflicts with an internal blame or something I could try and ensure we set this setting in all our repos instead","shortMessageHtmlLink":"Default to showing blame"}},{"before":"498f4eaa4e9022858ae8ec0f9f91872b8c712717","after":"9939f61a58df9b854298eee0a1ce93af3e8c0072","ref":"refs/heads/sapling-pr-archive-alex-statsig","pushedAt":"2024-01-18T17:38:33.000Z","pushType":"push","commitsCount":2907,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"merge commit for archive created by Sapling","shortMessageHtmlLink":"merge commit for archive created by Sapling"}},{"before":null,"after":"2006eef7853667e945d007d51182a51996753a28","ref":"refs/heads/pr816","pushedAt":"2024-01-18T17:38:27.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Add \"open in code review\" context menu action\n\nA common feature from other SCM extensions is a context-menu item to copy a link / open the selection/file in the remote SCM provider (ex. open in Github). This adds support for that in the editor / file explorer.\n\nCouple of things I wasn't totally confident about that I'd like thoughts on:\n- I opted to just add this to context menu in vscode since that's where most other extensions do this, but I'm guessing this might be desired in ISL too, which might changed some implementation details\n- Open to other titles for the commands, but afaict the message can't be dynamic (i.e. use code review provider's name), so I settled on \"Open in Code Review\"\n- I went with the simple approach of copying a link to the file on main using the same line numbers. This won't necessarily work if the file is dirty or modified in remote main. This could be mitigated by using the most recent public commit instead of main, and perhaps some magic with blame, but seemed more complex\n- Not sure the best way to grab the remote branch's name; just hard-coded \"main\" in my testing, but it could be \"master\" or something else as well. Is this easily retrievable from sapling?\n- I started setting a context for \"sapling:hasRepo\" so the option would only show if the editor has some folder with a sapling repo. I could be smarter and check if the active file is in a repo, but this seemed effective enough and simpler (scmProvider used in other commands is only available with the SCM panel)","shortMessageHtmlLink":"Add \"open in code review\" context menu action"}},{"before":"540197e86ecb8387c76e6bfc4e1d63a80ce00648","after":"25d6b85bb2a45f583cd1a689a60f3288b540e1ea","ref":"refs/heads/main","pushedAt":"2024-01-18T04:48:33.000Z","pushType":"push","commitsCount":2905,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"TopLevelToast: fine tune CSS\n\nSummary:\nApply CSS to the `AnimatedReorderGroup` `div` directly so we can use flexbox\nfor layout. This seems nicer since `translate` might render at sub-pixel\n(non-integer `px`) that looks blurry.\n\nReviewed By: evangrayk\n\nDifferential Revision: D52851255\n\nfbshipit-source-id: 0f6753a00bc7f124f4acc33006335feca85616de","shortMessageHtmlLink":"TopLevelToast: fine tune CSS"}},{"before":"52ac8ce77a949ed018e4ce45ba2565f823d5349e","after":"498f4eaa4e9022858ae8ec0f9f91872b8c712717","ref":"refs/heads/sapling-pr-archive-alex-statsig","pushedAt":"2023-08-28T21:54:29.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Merge 1882bd501374044e4f394b42c18352ba0ee8b94f into sapling-pr-archive-alex-statsig","shortMessageHtmlLink":"Merge <a class=\"commit-link\" data-hovercard-type=\"commit\" data-hovercard-url=\"https://github.com/alex-statsig/sapling/commit/1882bd501374044e4f394b42c18352ba0ee8b94f/hovercard\" href=\"https://github.com/alex-statsig/sapling/commit/1882bd501374044e4f394b42c18352ba0ee8b94f\"><tt>1882bd5</tt></a> into sapling-pr-archive-alex-statsig"}},{"before":null,"after":"1882bd501374044e4f394b42c18352ba0ee8b94f","ref":"refs/heads/pr718","pushedAt":"2023-08-28T21:54:26.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Add merge queue diff status","shortMessageHtmlLink":"Add merge queue diff status"}},{"before":"2f89f20a384305182367a25c62d417aded60be0c","after":"52ac8ce77a949ed018e4ce45ba2565f823d5349e","ref":"refs/heads/sapling-pr-archive-alex-statsig","pushedAt":"2023-08-28T21:30:10.000Z","pushType":"push","commitsCount":985,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Merge b96bb3629e5d7178b556863fda23f3392dcef5ed into sapling-pr-archive-alex-statsig","shortMessageHtmlLink":"Merge <a class=\"commit-link\" data-hovercard-type=\"commit\" data-hovercard-url=\"https://github.com/alex-statsig/sapling/commit/b96bb3629e5d7178b556863fda23f3392dcef5ed/hovercard\" href=\"https://github.com/alex-statsig/sapling/commit/b96bb3629e5d7178b556863fda23f3392dcef5ed\"><tt>b96bb36</tt></a> into sapling-pr-archive-alex-statsig"}},{"before":null,"after":"b96bb3629e5d7178b556863fda23f3392dcef5ed","ref":"refs/heads/pr717","pushedAt":"2023-08-28T21:30:07.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Add file decoration provider\n\nInspired by the built-in git extension's decoration provider, highlights changed/add files in the tabbar / file explorer (ex. by showing \"M\" for modified, and a yellow color). This came up as a gap from using git with vscode that led to people not adopting sapling.\n\n\nBased on: https://github.com/microsoft/vscode/blob/ec041fc442f37904e2f6608ee1f29257f474cf47/extensions/git/src/decorationProvider.ts#L166\n\nSee: https://github.com/microsoft/vscode-docs/blob/cd717bb4195d40ae8f556679348d38d6dce4321a/release-notes/v1_53.md#tab-decorations and https://stackoverflow.com/questions/66905124/vscode-how-to-remove-git-decorator-information-from-file-tabs","shortMessageHtmlLink":"Add file decoration provider"}},{"before":"8ec94b650e7506c8b6b359bf4291c93da37e2e63","after":"540197e86ecb8387c76e6bfc4e1d63a80ce00648","ref":"refs/heads/main","pushedAt":"2023-08-28T20:28:53.000Z","pushType":"push","commitsCount":780,"pusher":{"login":"alex-statsig","name":"Alex Coleman","path":"/alex-statsig","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/77301670?s=80&v=4"},"commit":{"message":"Updating submodules\n\nSummary:\nGitHub commits:\n\nhttps://github.com/facebook/fbthrift/commit/9a39d1c9dbd39123c96d324d37c530f1ec5a346b\nhttps://github.com/facebook/mvfst/commit/3b1ce5b8540186d112c48aba91f631d7cf6c3d54\nhttps://github.com/facebook/ocamlrep/commit/d63494539eab200b1f5bc5668906cd2ca3ab8c1f\nhttps://github.com/facebook/wangle/commit/d8ae7dfb4f58ec73fbd674876f6b6228c64d69f4\nhttps://github.com/facebookincubator/velox/commit/895291403ee2bb73ce2ba59ccc8052eadb7daeb5\n\nReviewed By: jailby\n\nfbshipit-source-id: c5910d86b43ba5feed5448d735d5de2bc02e0b73","shortMessageHtmlLink":"Updating submodules"}}],"hasNextPage":true,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"Y3Vyc29yOnYyOpK7MjAyNC0wNy0yNVQwNDo1NzoxOS4wMDAwMDBazwAAAASInGzP","startCursor":"Y3Vyc29yOnYyOpK7MjAyNC0wNy0yNVQwNDo1NzoxOS4wMDAwMDBazwAAAASInGzP","endCursor":"Y3Vyc29yOnYyOpK7MjAyMy0wOC0yOFQyMDoyODo1My4wMDAwMDBazwAAAAN0U5nG"}},"title":"Activity · alex-statsig/sapling"}