Skip to content

Conversation

@ultrox
Copy link
Contributor

@ultrox ultrox commented Mar 29, 2025

I think we can do bit better here by allowing people to copy/paste working code. Without imports, code is not working which could lead to some friction and confusion.

I think we can do bit better here by allowing people to copy/paste working code. Without imports, code is not working which could lead to some friction and confusion.
Copy link
Member

@lpil lpil left a comment

Choose a reason for hiding this comment

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

That's a syntax error! Please move the imports out of the functions 🙏

I also run this in the editor, so imports are sorted as well. There are few warnings though.
@ultrox
Copy link
Contributor Author

ultrox commented Mar 29, 2025

That's a syntax error! Please move the imports out of the functions 🙏

sorry about that, it should be fine now.

Copy link
Member

@lpil lpil left a comment

Choose a reason for hiding this comment

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

Thank you!!

@lpil lpil merged commit a18d7ff into gleam-lang:main Mar 29, 2025
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.

2 participants