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

Show objectscript.conn.docker-compose type connections under 'Current' node #251

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

Conversation

gjsjohnmurray
Copy link
Collaborator

This PR closes #187

For example:

image

This workspace consists of 2 local folders. Each contains a docker-compose.yml and a Dockerfile plus a .vscode/settings.json with an objectscript.conn referencing the corresponding Docker service.

The enhancement makes the two docker servers appear in the Current node of the tree. The (dynamic) local port for each is displayed.

Their namespaces can be browsed in Server Manager in the usual manner. Lite Terminals can be launched, and isfs or isfs-readonly folders can be added to the workspace.

This enhancement depends on a related change in the ObjectScript extension (intersystems-community/vscode-objectscript#1471)

The VSIX built from this PR should work OK with ObjectScript VSIXes that don't contain the relevant change, and vice versa. However the new features will only become available when both extensions have been updated.

@isc-bsaviano
Copy link
Contributor

I like the way this looks! I'll approve once the corresponding vscode-objectscript PR is approved and merged.

@gjsjohnmurray
Copy link
Collaborator Author

@isc-bsaviano and @isc-rsingh please try installing the VSIXes from both PRs to validate them.

@isc-rsingh
Copy link
Member

will try it out this weekend....

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.

Handle case where objectscript.conn.docker-compose is being used to deal with dynamic host port
3 participants