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

polywrap.app.yaml and polywrap.plugin.yaml should become polywrap.yaml #1295

Closed
pileks opened this issue Sep 30, 2022 · 0 comments · Fixed by #1296
Closed

polywrap.app.yaml and polywrap.plugin.yaml should become polywrap.yaml #1295

pileks opened this issue Sep 30, 2022 · 0 comments · Fixed by #1296

Comments

@pileks
Copy link
Contributor

pileks commented Sep 30, 2022

Is your feature request related to a problem? Please describe.
As part of the manifest refactor (#1057) we have virtually unified the Polywrap, App and Plugin manifests, with a few minor exceptions.
As part of those changes, we want to communicate that polywrap.yaml is the default filename for all project manifests (including app and plugin projects).

Describe the solution you'd like
The Polywrap CLI help files should only show polywrap.yaml | polywrap.yml | polywrap.json as the default project manifest files.
polywrap.app.* and polywrap.plugin.* should still be silently supported, but if they are detected, the user should be warned about them getting depecated in favor of polywrap.*.
All app and plugin project templates should also have their project manifest files renamed to polywrap.yaml.

Describe alternatives you've considered
A unification of the ...Manifest types has been considered (PluginManifest, AppManifest, PolywrapManifest).
This will not be considered within this issue, as the end-result of the issue should be a simple improvement to the quality of the experience a developer has when using Polywrap.

Additional context
/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants