Skip to content

feat: improve unexpected label error message#5329

Merged
lpil merged 6 commits into
gleam-lang:mainfrom
ankddev:improve-unexpected-label-error
Feb 7, 2026
Merged

feat: improve unexpected label error message#5329
lpil merged 6 commits into
gleam-lang:mainfrom
ankddev:improve-unexpected-label-error

Conversation

@ankddev

@ankddev ankddev commented Jan 28, 2026

Copy link
Copy Markdown
Contributor

  • The changes in this PR have been discussed beforehand in an issue
  • The issue for this PR has been linked
  • Tests have been added for new behaviour
  • The changelog has been updated for any user-facing changes

@giacomocavalieri giacomocavalieri left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a little comment inline!

Comment thread compiler-core/src/type_/pattern.rs Outdated

@giacomocavalieri giacomocavalieri left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@giacomocavalieri

Copy link
Copy Markdown
Member

Ah it seems this has a conflict with main, would you mind rebasing?

@ankddev

ankddev commented Jan 29, 2026

Copy link
Copy Markdown
Contributor Author

@giacomocavalieri Yeah, will rebase once I will be at home!

@ankddev
ankddev force-pushed the improve-unexpected-label-error branch from 2b367cc to df4a4cd Compare January 29, 2026 12:45

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one isn't subject of this PR, it was changed since I have runned Clippy

@ankddev

ankddev commented Jan 29, 2026

Copy link
Copy Markdown
Contributor Author

@giacomocavalieri Done!

@lpil lpil left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect, thank you!!

ankddev and others added 6 commits February 7, 2026 09:57
Signed-off-by: Andrey <andrekabatareika@gmail.com>
Signed-off-by: Andrey <andrekabatareika@gmail.com>
Signed-off-by: Andrey <andrekabatareika@gmail.com>
Signed-off-by: Andrey <andrekabatareika@gmail.com>
Signed-off-by: Andrey <andrekabatareika@gmail.com>
Co-authored-by: Giacomo Cavalieri <giacomo.cavalieri@icloud.com>
@lpil
lpil force-pushed the improve-unexpected-label-error branch from c7c2bdd to 499ec86 Compare February 7, 2026 09:59
@lpil
lpil enabled auto-merge (rebase) February 7, 2026 10:01
@lpil
lpil merged commit a7488e3 into gleam-lang:main Feb 7, 2026
12 checks passed
@ankddev
ankddev deleted the improve-unexpected-label-error branch February 7, 2026 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve error message on passing labeled arguments to a function with unlabeled arguments

3 participants