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.

Sign up for GitHub

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

Jump to bottom

Detect Apama, fix up settings, use stdio LSP, miscellaneous cleanup #63

Draft
wants to merge 22 commits into
base: master
Choose a base branch
from
Draft

Detect Apama, fix up settings, use stdio LSP, miscellaneous cleanup #63

wants to merge 22 commits into from

Conversation

bph-c8y
Copy link
Collaborator

@bph-c8y bph-c8y commented Jan 17, 2025 •
edited
Loading

  • Aim to automatically detect Apama installations.
  • Fix up settings to be properly rendered (i.e. use camelcase)
  • Use the new stdio interface
  • Other cleanup

TODO:

  • Make sure eplbuddy processes aren't being leaked. Relying on the VSCode LanguageClient to do so because it launches the process, but not entirely sure it is working.
  • Doc

bph-c8y added 22 commits October 23, 2024 11:30
@bph-c8y
Spike ExecutableResolver
86c26d3 
Improve our ability to find Apama processes on the system.
@bph-c8y
Begin to use ExecutableResolver within Extension
5242bf8
@bph-c8y
config: remove default apamahome, rationalise some descriptions
1f6aea1 
- Default apamahome assumed Windows.
- Various descriptions implied the set value was the default, which is obviously not true if it is configurable.
- Remove option for a "Remote" language server: while it currently works, it will soon be removed, and is antithetical to how MS expect you to run a Language Server.
@bph-c8y
settings: camel case for better VSCode rendering
9d311db 
VSCode automatically parses camel case names to be separate words.
@bph-c8y
make sure camelcase change is made in code
858006e
@bph-c8y
ExecutableResolver: cleaning up, adding a logger, fixing a bug
52d8d0d
@bph-c8y
extension: scope out roughly the logic to resolve apama_env on Unix
229e7f1
@bph-c8y
extension: roughly add back the automatic spawning logic
7d1b8fe
@bph-c8y
extension: remove some of the weird spawning logic
af1a3d8
@bph-c8y
Write up some Windows logic
dffc52f
@bph-c8y
Move release notes from README.md to GitHub Releases
17ae3ba
@bph-c8y
readme: more aggressive in requirements
a6a776b
@bph-c8y
README: more refinement
05fad7c
@bph-c8y
Don't resolve a failed path on failure
f158ef4
@bph-c8y
Use ExecutableResolver to find correlator, rather than LSP
10363a8 
We're going to use this to gate all advanced functionality.
@bph-c8y
Remove language server host option
445a545
@bph-c8y
extension: working spawn on Linux
c764e41
@bph-c8y
extension: experimental windows support
14e4316
@bph-c8y
extension: remove unused imports
5d14789
@bph-c8y
extension: some clean up, split up logging for language server
7b6aa1c
@bph-c8y
Attempt to clean up process (not working at the minute)
2858fce 
Also, some renaming of things to less generic names.
@bph-c8y
lsp: migrate from tcp to stdio
0aeed2f 
- Remove all networking artifacts.
- Let the Language Client handle spawning the process (and thus, process
	cleanup).
- Remove language server networking configuration options.
- Remove all process cleanup code, and associated libraries.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@bph-c8y