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

(GH-666) Puppetfile view #671

Merged
merged 1 commit into from
Jun 5, 2020

Conversation

jpogran
Copy link
Contributor

@jpogran jpogran commented May 28, 2020

Adds a new view to the Puppet toolbar view container that shows the puppet modules listed in the Puppetfile in the current workspace. This also adds a new command that will open the Puppetfile at the line of the selected module in the view.

Note: Needs an editor-services release with puppetlabs/puppet-editor-services#256

@jpogran jpogran added this to the 0.27.0 milestone May 28, 2020
@jpogran jpogran self-assigned this May 28, 2020
@jpogran jpogran force-pushed the gh-666-puppetfile-view branch 2 times, most recently from 0f5f73d to 9a19654 Compare June 1, 2020 18:13
@jpogran jpogran marked this pull request as ready for review June 1, 2020 18:14
@jpogran jpogran requested a review from glennsarti as a code owner June 1, 2020 18:14
Adds a new view to the Puppet toolbar view container that shows the puppet modules listed in the Puppetfile in the current workspace. This also adds a new command and context menu that will open the Puppetfile at the line of the selected module in the view.

This uses the `openTextDocument` VS Code API to open the Puppetfile without showing the file to the user. This initiates a `didOpen` notification which sends the content to the Language Server. If we opened the document and sent the content ourselves, a `didOpen` notitication is still sent, so it would result in sending the content twice.
@jpogran jpogran force-pushed the gh-666-puppetfile-view branch from 9a19654 to 1247c80 Compare June 5, 2020 01:35
@jpogran
Copy link
Contributor Author

jpogran commented Jun 5, 2020

0.26.1 editor-services is merged so this is clear to go

@jpogran jpogran merged commit 1075ca6 into puppetlabs:master Jun 5, 2020
@jpogran jpogran deleted the gh-666-puppetfile-view branch June 7, 2021 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant