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

langserver: Support --with-expectations-path option #892

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tk0miya
Copy link
Contributor

@tk0miya tk0miya commented Aug 6, 2023

Add --with-expectations=PATH option to the langserver subcommand. It suppresses warnings and info messages using expectations file like check command does. And it helps to users to develop RBS files gradually.

@tk0miya
Copy link
Contributor Author

tk0miya commented Aug 6, 2023

With this patch, many warnings/infos for the Steep project are suppressed on my VSCode.

Before:
スクリーンショット 2023-08-07 3 23 27

After:
スクリーンショット 2023-08-07 3 31 40

Some warnings are still shown. But it must be helpful to typing gradually :-)

Note: I noticed #468 had been filed after I created this patch. I'm unsure which is better to enable the "expectations" mode in the langserver. Please let me know if you feel other way is better. Thanks,

@@ -41,12 +42,13 @@ def self.start_worker(type, name:, steepfile:, steep_command:, index: nil, delay
steep_command: steep_command || "steep",
index: index,
delay_shutdown: delay_shutdown,
patterns: patterns
patterns: patterns,
with_expectations_path: with_expectations_path
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Note: I did not test the spawn_worker locally. How do I test this?

@soutaro soutaro added this to the Steep 1.6 milestone Oct 13, 2023
Add `--with-expectations=PATH` option to the langserver subcommand.
It suppresses warnings and info messages using expectations file like
check command does.  And it helps to users to develop RBS files
gradually.
@tk0miya tk0miya force-pushed the with-expectations-path_for_langserver branch from a19991c to 809ec80 Compare October 15, 2023 12:00
@soutaro soutaro removed this from the Steep 1.6 milestone Oct 27, 2023
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.

None yet

2 participants