Skip to content

fix: guide setup when envctl.yaml is missing - #47

Open
sam-bretz wants to merge 1 commit into
mainfrom
fix/missing-manifest-setup-guidance
Open

sam-bretz wants to merge 1 commit into
mainfrom
fix/missing-manifest-setup-guidance

Conversation

@sam-bretz

Copy link
Copy Markdown
Owner

Summary

  • report manifest presence and validity to the dashboard
  • replace the missing-manifest dead end with setup guidance and a copyable init command
  • disable New run until the repository is configured
  • add regression coverage for the missing-manifest state

Tests

  • focused Go regression test passed
  • node --check internal/web/assets/app.js passed
  • full web package tests were not runnable in the sandbox because loopback listeners are restricted

This branch has not been deployed

No deployments
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 this pull request may close these issues.

1 participant