-
Notifications
You must be signed in to change notification settings - Fork 524
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add Solang_Playground-milestone_1 #1126
Conversation
Signed-off-by: salaheldinsoliman <[email protected]>
Signed-off-by: salaheldinsoliman <[email protected]>
@keeganquigley Here it is! |
Signed-off-by: salaheldinsoliman <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the delivery @salaheldinsoliman I'm noticing the default deliverables (0a. - 0d.) are missing. I understand Docker and Article may not be applicable for this milestone but could you still add a Documentation/Testing sections with links? Thanks!
Please also see my evaluation; at the moment there are some unit tests failing. |
pinging @salaheldinsoliman |
@keeganquigley I'm currently working on a full-on test suit, that covers most of the functionality of the language server. I am also cleaning up the repo a little bit (removing warnings and improving documentation) |
@salaheldinsoliman sounds good thanks for the update! |
@keeganquigley Hello, With that being said, I request a re-evaluation of the delivery. |
Thanks for the changes @salaheldinsoliman looks good and I'm willing to accept the milestone. You can find my evaluation here. Looking forward to seeing compiling in the browser! Thanks for adding the test suite. |
Congratulations on completing the first milestone of this grant! As part of the Grants Program, we want to help grant recipients acknowledge their grants publicly. To that end, we've created a badge for projects that successfully deliver their first milestone. Please use the badge only in reference to the work that has been completed as part of this grant, so please do not display it on your team or project's homepage unless accompanied by a short description of the grant. Furthermore, you're now welcome to announce the grant publicly. Please remember to observe the foundation's guidelines in doing so. If you haven't already, reach out to [email protected] for feedback on your announcement and cross-promotion. |
🪙 Please fill out the invoice form in order to initiate the payment process. Thank you! |
@keeganquigley Thanks! |
@salaheldinsoliman, sorry for the delay. Your invoice should be processed on Monday latest. |
That's okay, thanks for the update!
…On Fri, Mar 8, 2024, 12:49 Sebastian Müller ***@***.***> wrote:
@salaheldinsoliman <https://github.com/salaheldinsoliman>, sorry for the
delay. Your invoice should be processed on Monday latest.
—
Reply to this email directly, view it on GitHub
<#1126 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AL4ZHS37KFN2GLTIZG5VHCLYXGQTLAVCNFSM6AAAAABCYL3X6KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBVGU2TOMRTHA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Milestone Delivery Checklist
Link to the application pull request: https://github.com/w3f/Grants-Program/pull/XXX < please fill this in with the PR number of your application.