Skip to content

Required property is missing: shell #377

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

Closed
ghost opened this issue Aug 20, 2024 · 0 comments
Closed

Required property is missing: shell #377

ghost opened this issue Aug 20, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@ghost
Copy link

ghost commented Aug 20, 2024

Describe the bug

The VS Code extension lacks critical linting support. Specifically, when creating a Composite Action, it fails to provide a warning that providing a shell property is required. I only became aware of this after pushing the update and encountering the following error:

.github/actions/aws-config/action.yml (Line: 18, Col: 7): Required property is missing: shell

To Reproduce
Steps to reproduce the behavior:

  1. With this workflow '...'
  2. Do this '...'
  3. See error

Expected behavior
VS Code extension shows a warning when authoring a Composite Action if the shell property isn't specified.

Screenshots
If applicable, add screenshots to help explain your problem.

Extension Version
v1.x.y

Additional context
Add any other context about the problem here.

@ghost ghost added the bug Something isn't working label Aug 20, 2024
@ghost ghost closed this as completed Aug 20, 2024
@github-project-automation github-project-automation bot moved this from Backlog 🗒 to Done 💤 in GitHub Actions VS Code Extension Aug 20, 2024
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

No branches or pull requests

0 participants