Skip to content
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

External evaluation of ink_analyzer milestone 3 #902

Merged
merged 2 commits into from
Jul 7, 2023

Conversation

dastansam
Copy link
Contributor

@dastansam dastansam commented Jul 1, 2023

In general, well documented and well-written code. There are some non-critical clippy warnings that can be addressed easily.

Looking forward to VSCode integration in the next milestone!

Link to the application pull request: w3f/Grants-Program#1615
Link to the milestone delivery: #900

@dastansam dastansam changed the title External evaluation of ink_analyzer Milestone 3 External evaluation of ink_analyzer milestone 3 Jul 1, 2023
@dastansam dastansam marked this pull request as ready for review July 1, 2023 19:50
@davidsemakula
Copy link
Contributor

davidsemakula commented Jul 2, 2023

@dastansam Thanks for the review and feedback! 🙂

There are some non-critical clippy warnings that can be addressed easily.

I've pushed style updates for inlining all format! arguments (among other things) so the non-critical clippy warnings should be resolved.

Looking forward to VSCode integration in the next milestone!

The VS Code extension should be ready by the end of this month (July), I'll make sure to ping you 🙂 .

@keeganquigley keeganquigley self-assigned this Jul 5, 2023
@keeganquigley
Copy link
Contributor

keeganquigley commented Jul 7, 2023

Thanks a bunch @dastansam great job as per usual. I'm happy to go ahead and accept it. I did have trouble running two tests, did you have any problems initially? See my comment here.

@keeganquigley keeganquigley merged commit de30a4c into w3f:master Jul 7, 2023
5 checks passed
@RouvenP
Copy link

RouvenP commented Jul 14, 2023

hi @dastansam we transferred the payment yesterday

@davidsemakula
Copy link
Contributor

@dastansam Just FYI 🙂
The VS Code extension is available at https://marketplace.visualstudio.com/items?itemName=ink-analyzer.ink-analyzer
Or you can get the VSIX directly from https://github.com/ink-analyzer/ink-vscode/releases

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.

4 participants