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

App manifest cleanups #1169

Open
9 of 25 tasks
noise64 opened this issue Dec 10, 2024 · 0 comments · May be fixed by #1194
Open
9 of 25 tasks

App manifest cleanups #1169

noise64 opened this issue Dec 10, 2024 · 0 comments · May be fixed by #1194
Assignees
Milestone

Comments

@noise64
Copy link
Contributor

noise64 commented Dec 10, 2024

  • break up app manifest parsing / validation into smaller parts
  • connected to the above: rework ValidationContext and / or Cow / sections
  • implement common custom command and clean
  • implement rmdirs
  • AppComponentContext: enable logging if text format is enabled
  • default profile is not overridable
  • deps up-to-date check: uses wrong input for checking if there were changes in deps
  • implement component (auto )selection and filter, and support it in:
    • component add
      • all, text mode only (or "streaming" json?)
      • if only one is auto selected, all modes
    • component update
      • all, text mode only (or "streaming" json?)
      • if only one is auto selected, all modes
    • app build
      • gen_rpc
      • componentize
      • link_rpc
      • clean
      • custom commands
  • improve the above auto select with "subfolder" support (and not just current dir)

Check:

  • wasmtools releases
  • coloring clashes between wasm rpc main / golem-cli main
  • "unknown field" field recommendation only show the extension properties
  • wac upstream?
@noise64 noise64 self-assigned this Dec 10, 2024
@vigoo vigoo added this to the Golem 1.2 milestone Dec 13, 2024
@noise64 noise64 linked a pull request Dec 15, 2024 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.

2 participants