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

Playbooks are not supported #6

Open
nkakouros opened this issue Mar 10, 2019 · 4 comments · May be fixed by #9
Open

Playbooks are not supported #6

nkakouros opened this issue Mar 10, 2019 · 4 comments · May be fixed by #9

Comments

@nkakouros
Copy link

I am trying to use this schema with yaml-language-server. After setting everything up, I create a new yml file and start editing it. I want this file to be a playbook:

- hosts: all
  tasks:
    - apt:

The first issue is that I get no completions for the tasks. Then, hosts complains that it needs a shell property. From what I understand the schema file supports only yml files with tasks and not playbooks. Is this the case? If so, can playbook support be added? Is there an existing workaround?

@shaded-enmity
Copy link
Owner

@tterranigma Functionality-wise work on #5 should enable this as well. The main problem here is to find reasonable pivot element (or a file name pattern) for which we can trigger this "enhanced" schema validation.

I might have some time to look into this on the weekend, will you be able to help me with testing once I have something testable?

@nkakouros
Copy link
Author

@shaded-enmity Sure! Just let me know when I can try it!

@bigjazzsound
Copy link

I'm also willing to test once this becomes available.

@nkakouros
Copy link
Author

@shaded-enmity Should you find the time to work on this during Christmas, I would be glad to test out.

shaded-enmity added a commit that referenced this issue Jan 3, 2020
- Enum values now allow string values as well to support templating
- Internal modules are not inspected using argspec
- Add support for playbooks (validates under the - task key)
- Add support for nested blocks (preliminary)

Fixes #6 #7 #8
@shaded-enmity shaded-enmity linked a pull request Jan 3, 2020 that will close this issue
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 a pull request may close this issue.

3 participants