chore: LTS backports #20855
Merged
chore: LTS backports #20855
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This includes: - scalameta/metals#5956 - scalameta/metals#5979 - scalameta/metals#5994 - scalameta/metals#5995 - scalameta/metals#5998 - scalameta/metals#5999 - scalameta/metals#6000 [Cherry-picked 7240327]
We now suggest to compile with -explain-cyclic, in which case we give a trace of the forcings that led to the cycle. The reason for the separate option is that maintaining a trace is not free so we should not be doing it by default. [Cherry-picked d2becfe]
[Cherry-picked 1f683cc]
[Cherry-picked 8d1d413]
[Cherry-picked e873376]
[Cherry-picked c6e103a]
[Cherry-picked abffedc]
…ary-cc` [Cherry-picked c5c5f92]
[Cherry-picked afb4825]
[Cherry-picked 075ee34][modified]
[Cherry-picked 69af22c]
Several improvements: - Follow opaque aliases as if they were aliases. Otherwise we risk crashing in phases where the `opaque` is dropped and these become regular aliases. - Update the `isInteresting` test to follow more types. Previously the test was too strict so some cycles were missed. - Make -explain-cyclic also explain for cycles detected by this check. Fixes #19372 [Cherry-picked 756ae34]
[Cherry-picked 1354d0d]
[Cherry-picked 882f2c7]
[Cherry-picked f8dd263]
* Disable `tests/neg/i8984.scala`. * Fix broken tests [Cherry-picked 92c5cb2][modified]
[Cherry-picked a0a8f0e]
[Cherry-picked a34cd7d]
[Cherry-picked a7f514a]
[Cherry-picked 7936264]
[Cherry-picked bc057db]
[Cherry-picked 9d1e5a6]
[Cherry-picked 44e63a1]
[Cherry-picked 96ec1aa][modified]
[Cherry-picked 69d1de3][modified]
…amed arg support (#19214) Previously, signature help had unstable way of calculating active parameter inside the signature help. It is now changed to work better with erroneous trees such as unclosed openings. It also adds a new feature for signature help, which will help user navigate inside named arguments. Furthermore, it will now find reordered named arguments, and display in the same way, by also marking the remaining parameters that they are now required to be named. Example: https://github.com/lampepfl/dotty/assets/48657087/b181d2d5-60f0-46a5-b2df-a58aa5f07454 The following changes have been made: - we will stop supporting Signature Helps for unclosed openings. It is too hard to recover from such errors reliably and to track where we are with the cursor. All modern editors will close opening on input. It simplified the code, and is the best way forward. - if there is an error within a definition return type parameter, we will now fall back to source, instead of print error, - tuples and functions are now supported when they are returned from methods. - this rewrite made me change a few things, and I took the chance and made it support clause interleaving. There is now a proper test suite for that. It required changing the way we represent Signatures. - argument reordering now works for all parameter lists, not only the currently applied one, - type parameters now also correctly display documentation I've changed the PR to use ShortenedTypePrinter and fixed some other minor issues like correctly marking reordered argumentss or hiding synthetic parameter names. In the future, Signature help should be changed to return symbols from the compiler instead of already printed values, and only print them later using shared API of ShortenedTypePrinter in the Presentation Compiler. Ideally we'd want a shared implementation for label printing with completions, hovers and other features. [Cherry-picked 01171de]
[Cherry-picked 38d252e]
Backports #19410 to the LTS branch. PR submitted by the release tooling. [skip ci]
) Backports #19408 to the LTS branch. PR submitted by the release tooling. [skip ci]
Backports #19429 to the LTS branch. PR submitted by the release tooling. [skip ci]
Backports #19186 to the LTS branch. PR submitted by the release tooling. [skip ci]
Backports #19432 to the LTS branch. PR submitted by the release tooling. [skip ci]
Backports #19337 to the LTS branch. PR submitted by the release tooling. [skip ci]
Backports #19459 to the LTS branch. PR submitted by the release tooling. [skip ci]
Backports #19453 to the LTS branch. PR submitted by the release tooling. [skip ci]
Backports #19452 to the LTS branch. PR submitted by the release tooling. [skip ci]
Backports #19465 to the LTS branch. PR submitted by the release tooling. [skip ci]
Backports #19561 to the LTS branch. PR submitted by the release tooling. [skip ci]
Backports #19428 to the LTS branch. PR submitted by the release tooling. [skip ci]
Backports #19471 to the LTS branch. PR submitted by the release tooling. [skip ci]
Backports #19488 to the LTS branch. PR submitted by the release tooling. [skip ci]
Backports #19484 to the LTS branch. PR submitted by the release tooling. [skip ci]
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.