Skip to content

Conversation

anatoliykmetyuk
Copy link
Contributor

No description provided.

@anatoliykmetyuk
Copy link
Contributor Author

Everyone involved, if you have a minute, please review:

@yuly16
@mkhan45
@rvacaru
@ShintaroSasaki

@noelwelsh
@bjaglin
@mlachkar
@tgodzik
@WojciechMazur

Copy link
Contributor

@darjutak darjutak left a comment

Choose a reason for hiding this comment

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

LGTM 👌

@mkhan45
Copy link

mkhan45 commented Nov 1, 2022

Hi, I'm glad to see this writeup! I think this line should be changed:

Mikail Khan, mentored by Noel Welsh, developed such a GUI during the Summer of 2022

Doodle Explore isn't a GUI, it's a specification for an eDSL that allows artists to create GUIs. Should I make a PR?

@WojciechMazur
Copy link
Contributor

LGTM

Copy link
Contributor

@julienrf julienrf 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 Anatolii for the clear article. I’ve left a few comments, suggestions, or questions.

@ShintaroSasaki
Copy link

LGTM

@anatoliykmetyuk anatoliykmetyuk marked this pull request as ready for review November 2, 2022 10:14
@noelwelsh
Copy link
Contributor

noelwelsh commented Nov 2, 2022 via email

Copy link
Contributor

@tgodzik tgodzik left a comment

Choose a reason for hiding this comment

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

One suggestion, otherwise LGTM!

@mkhan45
Copy link

mkhan45 commented Nov 3, 2022

Looks good, thanks again

Copy link
Contributor

@bjaglin bjaglin left a comment

Choose a reason for hiding this comment

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

Thanks @anatoliykmetyuk for coordinating the participation to the program!

@anatoliykmetyuk anatoliykmetyuk merged commit ff56938 into scala:main Nov 15, 2022
@anatoliykmetyuk anatoliykmetyuk deleted the gsoc2022-report branch November 15, 2022 09:55
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.

10 participants