dotnet / roslyn-tools

Tools used in Roslyn based repos
MIT License
102 stars 76 forks source link

Improve PR Descriptions #1425

Closed ryzngard closed 4 months ago

ryzngard commented 4 months ago

Use the GH Api to fetch PR titles if the commit message isn't indicative of a user provided message

TODO: This is limited by IP to 60 requests per hour. I managed to exceed that fairly easily.

diff --git a/description.md b/description.md
index 740b4d1..ae2cfc9 100644
--- a/description.md
+++ b/description.md
@@ -1,52 +1,52 @@
 [View Complete Diff of Changes](https://github.com/dotnet/roslyn/compare/dd3795a49875bef2728f01e0284406f33ea1e2e1...e5dc6c6e6afd5bd217309ee540a22e1d88489a52?w=1)
 - [Snap 17.11 Preview 2 (73753)](https://github.com/dotnet/roslyn/pull/73753)
-- [Merge pull request 73729 from CyrusNajmabadi/taggerAllocs](https://github.com/dotnet/roslyn/pull/73729)
+- [Lower allocations in tagging](https://github.com/dotnet/roslyn/pull/73729)
 - [Align sort implementation with reference from dotnet/runtime (73739)](https://github.com/dotnet/roslyn/pull/73739)
 - [Optimization of (de)serialization of ProjectId.DebugNames (73734)](https://github.com/dotnet/roslyn/pull/73734)
-- [Merge pull request 73727 from CyrusNajmabadi/arrayAlloc](https://github.com/dotnet/roslyn/pull/73727)
-- [Merge pull request 73724 from CyrusNajmabadi/vs4macApi](https://github.com/dotnet/roslyn/pull/73724)
-- [Merge pull request 73721 from CyrusNajmabadi/asyncAwaitPerf](https://github.com/dotnet/roslyn/pull/73721)
-- [Merge pull request 73723 from CyrusNajmabadi/codeDefWindow](https://github.com/dotnet/roslyn/pull/73723)
-- [Merge pull request 73722 from CyrusNajmabadi/braceMatcherPerf](https://github.com/dotnet/roslyn/pull/73722)
-- [Merge pull request 73719 from CyrusNajmabadi/stringIndentationRecurse](https://github.com/dotnet/roslyn/pull/73719)
+- [Avoid unnecessary array+linq allocs in common case](https://github.com/dotnet/roslyn/pull/73727)
+- [Remove legacy api used only by vs4mac.](https://github.com/dotnet/roslyn/pull/73724)
+- [Don't run code in the async/await highlighter unless on one of those keywords.](https://github.com/dotnet/roslyn/pull/73721)
+- [Switch to work queue for code definition window](https://github.com/dotnet/roslyn/pull/73723)
+- [Streamline brace matcher code](https://github.com/dotnet/roslyn/pull/73722)
+- [Small tweaks to the recursion walk we do when looking for strign indentation regions](https://github.com/dotnet/roslyn/pull/73719)
 - [Avoid unnecessary tree walk when hitting non-relevant directives (73720)](https://github.com/dotnet/roslyn/pull/73720)
-- [Merge pull request 73715 from CyrusNajmabadi/allIntervalTreeWork](https://github.com/dotnet/roslyn/pull/73715)
+- [Reduce lots of allocations in TagSpanIntervalTree](https://github.com/dotnet/roslyn/pull/73715)
 - [Cache the getnode/gettoken classifiers retrieived from the extension manager. (73712)](https://github.com/dotnet/roslyn/pull/73712)
-- [Merge pull request 73717 from CyrusNajmabadi/classificatoinStack](https://github.com/dotnet/roslyn/pull/73717)
-- [Merge pull request 73713 from CyrusNajmabadi/tagSpan](https://github.com/dotnet/roslyn/pull/73713)
-- [Merge pull request 73714 from CyrusNajmabadi/intervalTreeSpan](https://github.com/dotnet/roslyn/pull/73714)
-- [Merge pull request 73708 from CyrusNajmabadi/treeEfficiency](https://github.com/dotnet/roslyn/pull/73708)
+- [Don't push non-intersecting nodes/tokens to the classification stack](https://github.com/dotnet/roslyn/pull/73717)
+- [Move over to IDE using TagSpan uniformly](https://github.com/dotnet/roslyn/pull/73713)
+- [Move interval trees over to being TextSpan based](https://github.com/dotnet/roslyn/pull/73714)
+- [Pool collections during tagging.](https://github.com/dotnet/roslyn/pull/73708)
 - [Do not suppress the warning, which doesn't need to be suppressed (73711)](https://github.com/dotnet/roslyn/pull/73711)
 - [Reduce size of interval tree used for tagging (73703)](https://github.com/dotnet/roslyn/pull/73703)
-- [Merge pull request 73574 from RenderMichael/switch-or](https://github.com/dotnet/roslyn/pull/73574)
+- [Add support for or-pattern in `IDE0010`](https://github.com/dotnet/roslyn/pull/73574)
 - [Convert interval-tree type to a struct (73676)](https://github.com/dotnet/roslyn/pull/73676)
-- [Merge pull request 73699 from CyrusNajmabadi/lessMainThreadSwitch](https://github.com/dotnet/roslyn/pull/73699)
+- [Update tagging to avoid jumping back to the UI thread when finished.](https://github.com/dotnet/roslyn/pull/73699)
 - [Utilize ImmutableCollectionsMarshal to get ROS for XxHash128.Hash call (73692)](https://github.com/dotnet/roslyn/pull/73692)
 - [Return the SignatureHelp items nearest to the cursor (73606)](https://github.com/dotnet/roslyn/pull/73606)
 - [Final editor side changes for "allow ref struct" support (73700)](https://github.com/dotnet/roslyn/pull/73700)
-- [Merge pull request 73688 from CyrusNajmabadi/inProgress](https://github.com/dotnet/roslyn/pull/73688)
-- [Merge pull request 73673 from CyrusNajmabadi/lineCache](https://github.com/dotnet/roslyn/pull/73673)
+- [Ensure we sync source-generator versions over properly when doing a cone-sync](https://github.com/dotnet/roslyn/pull/73688)
+- [Reduce cost of syntactic classification by around 25% while paging down through a file (and around 50% while arrowing down).](https://github.com/dotnet/roslyn/pull/73673)
 - [Avoid having to go back to the UI thread in the navbar code (73681)](https://github.com/dotnet/roslyn/pull/73681)
 - [Include more data for debugging purposes (73684)](https://github.com/dotnet/roslyn/pull/73684)
-- [Merge pull request 73680 from CyrusNajmabadi/navBarCAdence](https://github.com/dotnet/roslyn/pull/73680)
+- [Slow down the work navbar does, and cancel updating it when receiving a high volume of user events](https://github.com/dotnet/roslyn/pull/73680)
 - [Expose XAML IDiagnosticSourceProvider names (73674)](https://github.com/dotnet/roslyn/pull/73674)
 - [Move EnC workspace tests down to Features layer (73660)](https://github.com/dotnet/roslyn/pull/73660)
 - [Strengthen language version tests for Ref Struct Interfaces (73607)](https://github.com/dotnet/roslyn/pull/73607)
 - [Implement ICollection<T> on ArrayBuilder (73659)](https://github.com/dotnet/roslyn/pull/73659)
 - [Add test for F1 on "allows ref struct" (73654)](https://github.com/dotnet/roslyn/pull/73654)
-- [Merge pull request 73648 from CyrusNajmabadi/classificationPerf](https://github.com/dotnet/roslyn/pull/73648)
+- [Drop roslyn impact on scrolling perf by 30%](https://github.com/dotnet/roslyn/pull/73648)
 - [Work around crash in Document Outline (73564)](https://github.com/dotnet/roslyn/pull/73564)
 - [Ignore drive casing when comparing on windows platforms (73380)](https://github.com/dotnet/roslyn/pull/73380)
-- [Merge pull request 73635 from CyrusNajmabadi/cacheServices](https://github.com/dotnet/roslyn/pull/73635)
-- [Merge pull request 73647 from CyrusNajmabadi/ignoreWhitespace](https://github.com/dotnet/roslyn/pull/73647)
-- [Merge pull request 73634 from CyrusNajmabadi/fileScopedNamespace](https://github.com/dotnet/roslyn/pull/73634)
-- [Merge pull request 73646 from CyrusNajmabadi/asNode2](https://github.com/dotnet/roslyn/pull/73646)
-- [Merge pull request 73645 from CyrusNajmabadi/asNode](https://github.com/dotnet/roslyn/pull/73645)
-- [Merge pull request 73629 from CyrusNajmabadi/classificationWalk](https://github.com/dotnet/roslyn/pull/73629)
+- [Cache classification services](https://github.com/dotnet/roslyn/pull/73635)
+- [Ignore the commits that update us to FileScopedNamespaces](https://github.com/dotnet/roslyn/pull/73647)
+- [Switch to file scoped namespace for tests](https://github.com/dotnet/roslyn/pull/73634)
+- [Simplify code in ForAttributeWithMetadataName](https://github.com/dotnet/roslyn/pull/73646)
+- [Switch to AsNode extension](https://github.com/dotnet/roslyn/pull/73645)
+- [Optimize syntactic classification walk](https://github.com/dotnet/roslyn/pull/73629)
 - [Add object browser support for ref struct interfaces (73621)](https://github.com/dotnet/roslyn/pull/73621)
 - [Remove unused usings (73633)](https://github.com/dotnet/roslyn/pull/73633)
 - [Update to simple using statements (73632)](https://github.com/dotnet/roslyn/pull/73632)
-- [Merge pull request 73625 from CyrusNajmabadi/constructorInit](https://github.com/dotnet/roslyn/pull/73625)
+- [Avoid searching entire trees when looking for implicit constructors in find-refs](https://github.com/dotnet/roslyn/pull/73625)
 - [Add signature help support for ref struct interfaces (73624)](https://github.com/dotnet/roslyn/pull/73624)
 - [Use semantics to check for readonly struct (72893)](https://github.com/dotnet/roslyn/pull/72893)
 - [Switch SG mode to 'balanced' by default (73618)](https://github.com/dotnet/roslyn/pull/73618)