-
Notifications
You must be signed in to change notification settings - Fork 13.9k
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
docs: update docker compose instructions #25479
Conversation
e1fc972
to
8f3c7ff
Compare
@mdeshmu I see this failed a check 😕 |
@sfirke pre-commit failure is not related to this PR.
|
Thanks for looking into the failed check! Unfortunately merging is blocked by CI rules until that unrelated thing is fixed. Either of us can close and re-open when we think that part of the build has been fixed, to re-trigger CI checks. And thanks for sending these multiple PRs, that's a great contribution 🙏 |
@mdeshmu would you mind providing links to the issues you mentioned to aid with context? |
@john-bodley before I write the same thing on the four different PRs, are you looking for the link to BugHerd or some other reference? |
This is ready to merge |
Co-authored-by: Sam Firke <sfirke@users.noreply.github.com>
Co-authored-by: Sam Firke <sfirke@users.noreply.github.com>
SUMMARY
Address bugherd task 69 & 73
cc: @sfirke @rusackas
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION