chore: Remove unused var notification_options #1237
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Removing the unused instance variable notification_options from the server. A new variable is instantiated and initialized in init but never used. Instead fresh notification_options object is accepted as an argument in the create_initialization_options and get_capabilities functions.
Motivation and Context
This removes confusion about the use of notification_options in the server.
How Has This Been Tested?
Existing tests work as expected
Breaking Changes
No
Types of changes
Checklist
Additional context
None