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

Reorganize tasks #87

Merged
merged 3 commits into from
Nov 8, 2023
Merged

Reorganize tasks #87

merged 3 commits into from
Nov 8, 2023

Conversation

Lee-W
Copy link
Collaborator

@Lee-W Lee-W commented Nov 7, 2023

  • Reorganize tasks and add ui server-related tasks. now it looks like the following
$ poetry run inv -l
Available tasks:

  api.init-poetry-env   Initialize the ask-astro API server local poetry environment
  api.run-with-docker   Run ask-astro API server with docker
  api.run-with-poetry   Run ask-astro API server with poetry
  api.stop-container    Stop ask-astro API server container
  docs.build            Build sphinx docs
  docs.serve            Serve the docs locally (http://127.0.0.1:8000)
  ui.init               Initialize UI server dependencies
  ui.run                Run UI server
  • fix doc missing links

fix #45

Copy link

cloudflare-pages bot commented Nov 7, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2cd6008
Status: ✅  Deploy successful!
Preview URL: https://19d055ae.ask-astro.pages.dev
Branch Preview URL: https://reorganize-tasks.ask-astro.pages.dev

View logs

Copy link
Collaborator

@sunank200 sunank200 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. But we should have building docs as part of CI @Lee-W

@Lee-W Lee-W added this to the 0.1.0 milestone Nov 8, 2023
@Lee-W Lee-W self-assigned this Nov 8, 2023
@Lee-W Lee-W merged commit 1d3d45d into main Nov 8, 2023
4 checks passed
@Lee-W Lee-W deleted the reorganize-tasks branch November 8, 2023 08:41
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.

Documentation for architecture for the deployment
2 participants