-
Notifications
You must be signed in to change notification settings - Fork 146
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
manifest.json was not found #5577
Comments
The manifest.json file must be at the top level, not in a subdirectory. |
@abyrne-moz Perhaps we should improve the error message to explicitly call this out, like: "manifest.json was not found at the top-level of the submitted file". |
FWIW this is already described in the full message (each linter notice/warning/error has a short message and a long description) - addons-linter/src/messages/layout.js Lines 35 to 41 in 883a496
|
Yes, and not everyone might click into the full report. |
It didn't help that on MacOS, you have to zip your files via the Terminal in order for the .zip file not to automatically create a new parent directory |
I would prefer to see this issue resolved by detecting if there was a single directory within the ZIP file with a valid manifest.json file and we just handle it correctly internally. |
I am told by the linter that my decompressed .zip file doesn't contain a manifest.json file. But it does.
The text was updated successfully, but these errors were encountered: