Fixed mount_be stdout.

[ericbsd]

Summary by Sourcery

This pull request addresses a bug in the mount_be function by ensuring that the stdout of the subprocess is captured.

• Bug Fixes:
  • Fixed the issue where the stdout of the mount_be function was not being captured.

[sourcery-ai]

Reviewer's Guide by Sourcery

This pull request fixes the handling of stdout in the 'mount_be' function by adding 'stdout=PIPE' to the 'run' function call. This change ensures that the standard output of the 'bectl' command is captured correctly.

---

Tips

• Trigger a new Sourcery review by commenting @sourcery-ai review on the pull request.
• Continue your discussion with Sourcery by replying directly to review comments.
• You can change your review settings at any time by accessing your dashboard:
  • Enable or disable the Sourcery-generated pull request summary or reviewer's guide;
  • Change the review language;
• You can always contact us if you have any questions or feedback.

[sourcery-ai]

Hey @ericbsd - I've reviewed your changes - here's some feedback:

Overall Comments:

• Consider capturing stderr as well as stdout to ensure all output from the subprocess is captured.
• The function returns a string, but the captured output isn't being used. Consider updating the function to return the captured output if that's the intended behavior.

Here's what I looked at during the review

• 🟢 General issues: all looks good
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.}