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

Support document symbols for shared examples #56

Merged
merged 5 commits into from
Jan 25, 2025

Conversation

byplayer
Copy link
Contributor

@byplayer byplayer commented Dec 5, 2024

I would like to support shared_examples, shared_context, shared_examples_for in document index.
This is official document about shared-examples

@st0012 st0012 changed the title Support shares Support document symbols for shared examples Dec 9, 2024
@st0012
Copy link
Owner

st0012 commented Dec 9, 2024

I tested this against sentry-ruby, which has quite a few shared_examples in its specs. But with this branch, document symbols in those files (example) mostly disappeared. Can you investigate it?

@byplayer
Copy link
Contributor Author

スクリーンショット 2024-12-29 18 40 44

It looks OK for me.
I checked all examples and contexts shown in the "OUTLINE" pane.

Could you check it again?

I found that "it_behaves_like" may be document-like example. I will fix it next pull request.

@st0012 st0012 added the enhancement New feature or request label Jan 25, 2025
Copy link
Owner

@st0012 st0012 left a comment

Choose a reason for hiding this comment

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

Sorry, it's probably my editor messing up the result at that time. Everything looks good now! Thanks for adding this support ❤️

@st0012 st0012 merged commit f2e4170 into st0012:main Jan 25, 2025
3 checks passed
@st0012
Copy link
Owner

st0012 commented Jan 25, 2025

I've released it in https://github.com/st0012/ruby-lsp-rspec/releases/tag/v0.1.21

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants