Update yml file. Add fixes to remove compiler warnings.

Author: PaulZC

.github/workflows/compile-sketch.yml

@@ -27,20 +27,29 @@ jobs:
             platforms: |
               - name: esp32:esp32
                 source-url: https://raw.githubusercontent.com/espressif/arduino-esp32/gh-pages/package_esp32_index.json
+                env:
+                  CS: 1
+                  LED_BUILTIN: 13
 
           # ESP32-S2
           # https://github.com/espressif/arduino-esp32/blob/master/boards.txt
           - fqbn: esp32:esp32:esp32s2
             platforms: |
               - name: esp32:esp32
                 source-url: https://raw.githubusercontent.com/espressif/arduino-esp32/gh-pages/package_esp32_index.json
+                env:
+                  CS: 1
+                  LED_BUILTIN: 13
 
           # ESP32-C3
           # https://github.com/espressif/arduino-esp32/blob/master/boards.txt
           - fqbn: esp32:esp32:esp32c3
             platforms: |
               - name: esp32:esp32
                 source-url: https://raw.githubusercontent.com/espressif/arduino-esp32/gh-pages/package_esp32_index.json
+                env:
+                  CS: 1
+                  LED_BUILTIN: 13
 
           # Artemis / Apollo3
           # https://github.com/sparkfun/Arduino_Apollo3/blob/main/boards.txt
@@ -55,6 +64,8 @@ jobs:
             platforms: |
               - name: esp8266:esp8266
                 source-url: https://arduino.esp8266.com/stable/package_esp8266com_index.json
+                env:
+                  CS: 1
 
           # SAMD21
           # https://github.com/arduino/ArduinoCore-samd/blob/master/boards.txt
@@ -97,6 +108,16 @@ jobs:
             - source-url: https://github.com/${{github.repository}}.git
           sketch-paths: |
             - ./
+            - '!./examples/Example11_ResetModule/Example2_FactoryDefaultsviaSerial/**'
+            - '!./examples/Example12_UseUart/**'
+            - '!./examples/Example13_PVT/Example3_AutoPVTviaUart/**'
+            - '!./examples/Example13_PVT/Example4_AssumeAutoPVTviaUart/**'
+            - '!./examples/Example2_NMEAParsing/**'
+            - '!./examples/Example8_GetProtocolVersion_Serial/**'
+            - '!./examples/NEO-M8P-2/Example3_BaseWithLCD/**'
+            - '!./examples/Data_Logging/**'
+            - '!./examples/ZED-F9P/Example14_NTRIPServer/**'
+            - '!./examples/ZED-F9P/Example4_BaseWithLCD/**'
           enable-warnings-report: true
           # verbose: true
 

examples/Example8_GetProtocolVersion_Serial/Example8_GetProtocolVersion_Serial.ino

@@ -31,9 +31,11 @@
 
 #include <SoftwareSerial.h>
 
-#define mySerial Serial1 // Uncomment this line to connect via Serial1
+//#define mySerial Serial1 // Uncomment this line to connect via Serial1
 // - or -
 //SoftwareSerial mySerial(10, 11); // Uncomment this line to connect via SoftwareSerial(RX, TX). Connect pin 10 to GNSS TX pin.
+// - or -
+#define mySerial Serial // Uncomment this line if you just want to keep using Serial
 
 #include <SparkFun_u-blox_GNSS_Arduino_Library.h> //http://librarymanager/All#SparkFun_u-blox_GNSS
 SFE_UBLOX_GNSS myGNSS;

src/SparkFun_u-blox_GNSS_Arduino_Library.cpp

@@ -1648,16 +1648,17 @@ void SFE_UBLOX_GNSS::processRTCMframe(uint8_t incoming)
 //This function is called for each byte of an RTCM frame
 //Ths user can overwrite this function and process the RTCM frame as they please
 //Bytes can be piped to Serial or other interface. The consumer could be a radio or the internet (Ntrip broadcaster)
-void SFE_UBLOX_GNSS::processRTCM(uint8_t incoming) // IGNORE COMPILER WARNING unused parameter 'incoming'
+void SFE_UBLOX_GNSS::processRTCM(uint8_t incoming)
 {
+  uint8_t ignoreMe = incoming; // Do something with incoming just to get rid of the pesky compiler warning!
+
   //Radio.sendReliable((String)incoming); //An example of passing this byte to a radio
 
   //_debugSerial->write(incoming); //An example of passing this byte out the serial port
 
   //Debug printing
   //  _debugSerial->print(F(" "));
   //  if(incoming < 0x10) _debugSerial->print(F("0"));
-  //  if(incoming < 0x10) _debugSerial->print(F("0"));
   //  _debugSerial->print(incoming, HEX);
   //  if(rtcmFrameCounter % 16 == 0) _debugSerial->println();
 }
@@ -2933,6 +2934,8 @@ sfe_ublox_status_e SFE_UBLOX_GNSS::sendCommand(ubxPacket *outgoingUBX, uint16_t
 //Returns false if sensor fails to respond to I2C traffic
 sfe_ublox_status_e SFE_UBLOX_GNSS::sendI2cCommand(ubxPacket *outgoingUBX, uint16_t maxWait)
 {
+  uint16_t ignoreMe = maxWait; // Do something with maxWait just to avoid the pesky compiler warnings!
+  
   // From the integration guide:
   // "The receiver does not provide any write access except for writing UBX and NMEA messages to the
   //  receiver, such as configuration or aiding data. Therefore, the register set mentioned in section Read
@@ -10735,8 +10738,10 @@ uint16_t SFE_UBLOX_GNSS::getMagAcc(uint16_t maxWait)
 }
 
 // getGeoidSeparation is currently redundant. The geoid separation seems to only be provided in NMEA GGA and GNS messages.
-int32_t SFE_UBLOX_GNSS::getGeoidSeparation(uint16_t maxWait) // IGNORE COMPILER WARNING unused parameter 'maxWait'
+int32_t SFE_UBLOX_GNSS::getGeoidSeparation(uint16_t maxWait)
 {
+  uint16_t ignoreMe = maxWait; // Do something with maxWait just to get rid of the pesky compiler warning
+
   return (0);
 }