Skip to content

Conversation

@caryoscelus
Copy link
Contributor

No description provided.

@Anton-4
Copy link
Collaborator

Anton-4 commented Nov 27, 2023

Thanks for helping out @caryoscelus! Could you add a test for this as well?

@caryoscelus
Copy link
Contributor Author

@Anton-4 sure! it seems there's also a lot of space to lower code repetition while i'm at it

@caryoscelus caryoscelus force-pushed the app-provides-error-reporting branch from 3929dd6 to 3332503 Compare November 28, 2023 02:19
@caryoscelus
Copy link
Contributor Author

there's also a lot of space to lower code repetition while i'm at it

on second thought, probably condensing code isn't worth here

@Anton-4 ready for review

Signed-off-by: Anton-4 <17049058+Anton-4@users.noreply.github.com>
Signed-off-by: Anton-4 <17049058+Anton-4@users.noreply.github.com>
Copy link
Collaborator

@Anton-4 Anton-4 left a comment

Choose a reason for hiding this comment

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

Looks good, thanks @caryoscelus!

@Anton-4 Anton-4 enabled auto-merge November 28, 2023 10:43
@Anton-4 Anton-4 merged commit 10897ab into roc-lang:main Nov 28, 2023
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.

None yet

2 participants