fix CI issues

esphome/components/i2s_audio/media_player/__init__.py

@@ -1,17 +1,15 @@
-import esphome.codegen as cg
-from esphome.components import media_player, esp32
-import esphome.config_validation as cv
-
 from esphome import pins
-
+import esphome.codegen as cg
+from esphome.components import esp32, media_player
+import esphome.config_validation as cv
 from esphome.const import CONF_ID, CONF_MODE
 
 from .. import (
-    i2s_audio_ns,
-    I2SAudioComponent,
-    I2SAudioOut,
     CONF_I2S_AUDIO_ID,
     CONF_I2S_DOUT_PIN,
+    I2SAudioComponent,
+    I2SAudioOut,
+    i2s_audio_ns,
 )
 
 CODEOWNERS = ["@jesserockz"]
@@ -130,7 +128,7 @@ async def to_code(config):
 
     cg.add_library(
         name="ESP32-audioI2S",
-        version=None,
         # use close-to latest commit, since tagged versions are fairly irregular...
-        repository="https://github.com/schreibfaul1/ESP32-audioI2S.git#1bc79e547ebb6f917bf82b47bec9b7e6a9b7e314",
+        repository="https://github.com/schreibfaul1/ESP32-audioI2S.git#",
+        version="1bc79e547ebb6f917bf82b47bec9b7e6a9b7e314",
     )

esphome/components/i2s_audio/media_player/i2s_audio_media_player.cpp

@@ -263,7 +263,7 @@ void I2SAudioMediaPlayer::dump_config() {
 #endif
 }
 
-bool I2SAudioMediaPlayer::connecttouri_(const std::string uri) {
+bool I2SAudioMediaPlayer::connecttouri_(const std::string &uri) {
   if (this->audio_ == nullptr) {
     return false;
   }

esphome/components/i2s_audio/media_player/i2s_audio_media_player.h

@@ -59,7 +59,7 @@ class I2SAudioMediaPlayer : public Component, public media_player::MediaPlayer,
   void stop_();
   void play_();
 
-  bool connecttouri_(const std::string uri);
+  bool connecttouri_(const std::string &uri);
 
   I2SState i2s_state_{I2S_STATE_STOPPED};
   std::unique_ptr<Audio> audio_;

platformio.ini

@@ -124,14 +124,13 @@ lib_deps =
     HTTPClient                           ; http_request,nextion (Arduino built-in)
     ESPmDNS                              ; mdns (Arduino built-in)
     DNSServer                            ; captive_portal (Arduino built-in)
-    esphome/ESP32-audioI2S@2.0.7         ; i2s_audio
+    https://github.com/schreibfaul1/ESP32-audioI2S.git#1bc79e547ebb6f917bf82b47bec9b7e6a9b7e314 ; i2s_audio
     droscy/esp_wireguard@0.4.2          ; wireguard
 
 build_flags =
     ${common:arduino.build_flags}
     -DUSE_ESP32
     -DUSE_ESP32_FRAMEWORK_ARDUINO
-    -DAUDIO_NO_SD_FS                ; i2s_audio
 extra_scripts = post:esphome/components/esp32/post_build.py.script
 
 ; This are common settings for the ESP32 (all variants) using IDF.