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

v2.17.0 errors out for data map type #187

Closed
jens-maus opened this issue Sep 30, 2024 · 1 comment · Fixed by #188
Closed

v2.17.0 errors out for data map type #187

jens-maus opened this issue Sep 30, 2024 · 1 comment · Fixed by #188

Comments

@jens-maus
Copy link

Since version v2.17.0 the addon-linter returns with the following error:

Error: [{'type': 'data', 'read_only': False, 'path': '/usr/local'}, {'type': 'backup', 'read_only': False}] is not valid under any of the given schemas

I suspect this is due to the missing data type not being part of the newer map linter check added in v2.17.0. However, due to the addon config documentation, data is a valid map. See here:

Bildschirmfoto 2024-09-30 um 08 43 26

Thus, I think data is simply missing as a valid map type in the latest linter configs.

So please consider adding data for action-addon-linter to not complain for addons defining this map type. See here for an actual addon that uses/defines this map:

https://github.com/jens-maus/RaspberryMatic/blob/master/home-assistant-addon/config.yaml#L17-L22

Revertiing to v2.16.0 immediately solves the issue.

@frenck
Copy link
Owner

frenck commented Sep 30, 2024

Thanks for report! That was indeed missed. Fix in https://github.com/frenck/action-addon-linter/releases/tag/v2.17.1

@github-actions github-actions bot locked and limited conversation to collaborators Oct 30, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants