search

MycroftAI / mycroft-core

Mycroft Core, the Mycroft Artificial Intelligence platform.
https://mycroft.ai
Apache License 2.0
6.51k stars 1.27k forks source link

Fix shellcheck error on Arch install command #3092

Closed krisgesling closed 2 years ago

krisgesling commented 2 years ago

Description

Fixes shellcheck error from PR #3091

How to test

Shellcheck wasn't running on dev_setup.sh when those tests ran. It is now so automated tests should pass.

Contributor license agreement signed?

codecov-commenter commented 2 years ago

Codecov Report

Merging #3092 (099d2a4) into dev (2d15fca) will not change coverage. The diff coverage is n/a.

:exclamation: Current head 099d2a4 differs from pull request most recent head be0e0d5. Consider uploading reports for the commit be0e0d5 to get more accurate results

@@           Coverage Diff           @@
##              dev    #3092   +/-   ##
=======================================
  Coverage   53.58%   53.58%           
=======================================
  Files         121      121           
  Lines       11128    11128           
=======================================
  Hits         5963     5963           
  Misses       5165     5165

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 2d15fca...be0e0d5. Read the comment docs.

devops-mycroft commented 2 years ago

Voight Kampff Integration Test Succeeded (Results)

krisgesling commented 2 years ago

Self-merging as it will cause everything else to fail.