Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

test: Inlay Hints #206

Merged
merged 4 commits into from
Feb 14, 2025
Merged

test: Inlay Hints #206

merged 4 commits into from
Feb 14, 2025

Conversation

xpyctumo
Copy link
Contributor

@xpyctumo xpyctumo commented Feb 14, 2025 •
edited
Loading

Resolves #201

Added tests for inlay hints in LSP: contract, traits, messages

Summary:

  • Added 19 tests for inlay hints
    • Error codes
    • Show gas consumption hints for asm functions
    • Show parameter name hints in function calls
    • Show exit codes in require() call
    • Show method ID in contract

@xpyctumo xpyctumo added tests All related to testing of the project lsp: inlayHint `textDocument/` Inlay Hint Request labels Feb 14, 2025
i582
i582 approved these changes Feb 14, 2025
View reviewed changes
Copy link
Collaborator

@i582 i582 left a comment

Choose a reason for hiding this comment

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

Great!

@i582 i582 merged commit d1ca640 into master Feb 14, 2025
4 checks passed
@i582 i582 deleted the inlay_hints_tests branch February 14, 2025 15:28
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@i582 i582 i582 approved these changes

Assignees
No one assigned
Labels
lsp: inlayHint `textDocument/` Inlay Hint Request tests All related to testing of the project
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Test inlay hints
2 participants
@xpyctumo @i582