Skip to content

Improve error message for custom type with fields but no constructor#5964

Merged
lpil merged 4 commits into
gleam-lang:mainfrom
giacomocavalieri:fix-5907
Jul 14, 2026
Merged

Improve error message for custom type with fields but no constructor#5964
lpil merged 4 commits into
gleam-lang:mainfrom
giacomocavalieri:fix-5907

Conversation

@giacomocavalieri

@giacomocavalieri giacomocavalieri commented Jul 11, 2026

Copy link
Copy Markdown
Member

This PR fixes #5907, this PR fixes #5377

  • 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

@ankddev

ankddev commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

This PR seems to fix #5377 too!

@giacomocavalieri

Copy link
Copy Markdown
Member Author

Aha, nice catch!

@giacomocavalieri giacomocavalieri changed the title Fix 5907 Improve error message for custom type with fields but no constructor Jul 11, 2026

@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.

Lovely! Thank you!!!

@lpil
lpil merged commit 832ebf8 into gleam-lang:main Jul 14, 2026
18 checks passed
@giacomocavalieri
giacomocavalieri deleted the fix-5907 branch July 14, 2026 14:35
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.

Provide uppercase hint for type variant names Report error about wrong C-style type definition for all fields

3 participants