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

Update qbit-manage.md to correct instances of qbit_manage to qbit-manage #320

Merged
merged 4 commits into from
Nov 30, 2024

Conversation

rsimplicio
Copy link
Contributor

@rsimplicio rsimplicio commented Nov 27, 2024

Several commands and the docker container name use a hyphen/dash rather than an underscore as the separator between qbit and manage. Updated instances where this caused an issue while following the documentation.

Please describe the purpose of this Pull Request.

Update erroneous instances where commands, container name, and paths use qbit_manage rather than what is actually created by the role, qbit-manage. Also found an issue with the logs directory missing from the tail command.

For new Sandbox app/role documentation, please confirm you have completed the following tasks:

  • [] App documentation page created
  • [] markdownlint reports no errors on edited page(s) - use https://dlaa.me/markdownlint/ if your IDE does not have this built in
  • [] Reference added in docs/sandbox/index.md
  • [] Reference added in mkdocs.yml nav menu

Several commands and the docker container name use a hyphen/dash rather than an underscore as the separator between qbit and manage. Updated instances where this caused an issue while following the documentation.
@saltydk saltydk requested a review from owine November 27, 2024 01:19
Copy link
Contributor

Deploying with ⚡ Cloudflare Pages

Latest commit:89088e9f78512b136aca6239f74b12a1de83e82b
Status: ✅  Deploy successful!
Preview URL:https://a59db747.docs-acq.pages.dev
Branch Preview URL:https://pr-320.docs-acq.pages.dev

@saltydk saltydk merged commit e623da9 into saltyorg:main Nov 30, 2024
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.

2 participants