Skip to content
This repository has been archived by the owner on Jul 31, 2023. It is now read-only.

Fix highlighting of method parameters. Add support for multiline parameters. #664

Merged
merged 1 commit into from
Oct 15, 2020

Conversation

graceful-potato
Copy link
Contributor

Fix for #663

Before
After

No more leaking scopes
Screenshot from 2020-10-11 20-00-50

Also added double splat operator.

  • The build passes
  • TSLint is mostly happy
  • Prettier has been run

@codecov
Copy link

codecov bot commented Oct 11, 2020

Codecov Report

Merging #664 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #664   +/-   ##
======================================
  Coverage    5.08%   5.08%           
======================================
  Files          10      10           
  Lines         118     118           
  Branches       20      20           
======================================
  Hits            6       6           
  Misses        112     112           
Flag Coverage Δ
#language_server_ruby 5.08% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2449c01...2d66c5e. Read the comment docs.

@wingrunr21
Copy link
Collaborator

Are there tests that could help with this for the future?

@graceful-potato
Copy link
Contributor Author

Created file methods.rb with tests

@graceful-potato graceful-potato force-pushed the iss663 branch 2 times, most recently from a25c8e6 to 15fb20d Compare October 12, 2020 20:45
feat: add support for multiline arguments
feat: add double splat operator
test: add tests
@wingrunr21 wingrunr21 merged commit cace1a3 into rubyide:master Oct 15, 2020
@wingrunr21 wingrunr21 mentioned this pull request Oct 15, 2020
3 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants