Merge branch 'master' into feature/ppp_after_linter

Author: me-no-dev

Diff for: .github/workflows/pre-commit-status.yml

@@ -0,0 +1,41 @@
+name: Report Pre-commit Check Status
+
+on:
+  pull_request_target:
+    types: [opened, reopened, synchronize, labeled, unlabeled]
+
+permissions:
+  statuses: write
+
+jobs:
+  report-run:
+    name: Check if the PR has run the pre-commit checks
+    runs-on: ubuntu-latest
+    steps:
+      - name: Report pending
+        uses: conda/actions/[email protected]
+        with:
+          context: "Pre-commit checks"
+          state: pending
+          description: The pre-commit checks need to be successful before merging
+
+      - name: Wait for pre-commit checks to complete
+        uses: lucasssvaz/wait-on-workflow@v1
+        if: |
+          contains(github.event.pull_request.labels.*.name, 'Status: Pending Merge')
+        id: wait-on-workflow
+        with:
+          timeout: 10
+          interval: 30
+          workflow: pre-commit.yml
+          sha: ${{ github.event.pull_request.head.sha || github.sha }}
+
+      - name: Report success
+        uses: conda/actions/[email protected]
+        if: |
+          contains(github.event.pull_request.labels.*.name, 'Status: Pending Merge') &&
+          steps.wait-on-workflow.outputs.conclusion == 'success'
+        with:
+          context: "Pre-commit checks"
+          state: success
+          description: All pre-commit checks passed

Diff for: .github/workflows/pre-commit.yml

@@ -1,14 +1,14 @@
-name: Pre-commit check
+name: Pre-commit hooks
 
 on:
   pull_request:
-    types: [opened, reopened, synchronize, labeled]
+    types: [opened, reopened, synchronize, labeled, unlabeled]
 
 jobs:
   lint:
     if: |
       contains(github.event.pull_request.labels.*.name, 'Status: Pending Merge')
-    name: Checking if any fixes are needed
+    name: Check if fixes are needed
     runs-on: ubuntu-latest
     steps:
       - name: Checkout latest commit

Diff for: .github/workflows/publishlib.yml

@@ -11,7 +11,7 @@ env:
   # It's convenient to set variables for values used multiple times in the workflow
   SKETCHES_REPORTS_PATH: artifacts/libraries-report
   GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}
-  PR_EVENT_PATH: artifacts/Event File/event.json
+  PR_NUMBER: ${{ github.event.workflow_run.pull_requests[0].number }}
 
 jobs:
   lib-test-results:
@@ -25,17 +25,22 @@ jobs:
       - name: Download and Extract Artifacts
         run: |
           mkdir -p artifacts && cd artifacts
+          mkdir -p libraries-report
           artifacts_url=${{ github.event.workflow_run.artifacts_url }}
           gh api "$artifacts_url" -q '.artifacts[] | [.name, .archive_download_url] | @tsv' | while read artifact
           do
             IFS=$'\t' read name url <<< "$artifact"
             gh api $url > "$name.zip"
-            unzip -d "$name" "$name.zip"
+            unzip -j "$name.zip" -d "temp_$name"
+            mv "temp_$name"/* libraries-report
+            rm -r "temp_$name"
           done
+          echo "Contents of parent directory:"
+          ls -R ..
     
       - name: Report results
-        uses: P-R-O-C-H-Y/report-size-deltas@main
+        uses: P-R-O-C-H-Y/report-size-deltas@libs
         with:
           sketches-reports-source: ${{ env.SKETCHES_REPORTS_PATH }}
           github-token: ${{ env.GITHUB_TOKEN }}
-          pr-event-path: ${{ env.PR_EVENT_PATH }}
+          pr-number: ${{ env.PR_NUMBER }}

Diff for: .github/workflows/publishsizes.yml

@@ -0,0 +1,54 @@
+name: Sizes Results
+
+on:
+  workflow_run:
+    workflows: [ESP32 Arduino CI]
+    types:
+      - completed
+
+  workflow_dispatch:
+env:
+  # It's convenient to set variables for values used multiple times in the workflow
+  SKETCHES_REPORTS_PATH: artifacts/sizes-report
+  GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}
+  PR_NUMBER: ${{ github.event.workflow_run.pull_requests[0].number }}
+
+jobs:
+  sizes-test-results:
+    name: Sizes Comparsion Results
+    runs-on: ubuntu-latest
+    if: |
+      github.event.workflow_run.event == 'pull_request' &&
+      github.event.workflow_run.conclusion == 'success'
+
+    steps:
+      - name: Download and Extract Artifacts
+        run: |
+          mkdir -p artifacts && cd artifacts
+          mkdir -p sizes-report
+          mkdir -p sizes-report/master
+          mkdir -p sizes-report/pr
+          artifacts_url=${{ github.event.workflow_run.artifacts_url }}
+          gh api "$artifacts_url" -q '.artifacts[] | [.name, .archive_download_url] | @tsv' | while read artifact
+          do
+            IFS=$'\t' read name url <<< "$artifact"
+            gh api $url > "$name.zip"
+            unzip -j "$name.zip" -d "temp_$name"
+            if [[ "$name" == *"master"* ]]; then
+              mv "temp_$name"/* sizes-report/master
+            elif [[ "$name" == *"pr"* ]]; then
+              mv "temp_$name"/* sizes-report/pr
+            else
+              mv "temp_$name"/* sizes-report
+            fi
+            rm -r "temp_$name"
+          done
+          echo "Contents of parent directory:"
+          ls -R ..
+    
+      - name: Report results
+        uses: P-R-O-C-H-Y/report-size-deltas@sizes_v2
+        with:
+          sketches-reports-source: ${{ env.SKETCHES_REPORTS_PATH }}
+          github-token: ${{ env.GITHUB_TOKEN }}
+          pr-number: ${{ env.PR_NUMBER }}

Diff for: cores/esp32/chip-debug-report.cpp

@@ -193,6 +193,7 @@ static void printPartitionsInfo(void) {
             case ESP_PARTITION_SUBTYPE_DATA_ESPHTTPD: chip_report_printf("ESPHTTPD"); break;
             case ESP_PARTITION_SUBTYPE_DATA_FAT: chip_report_printf("FAT"); break;
             case ESP_PARTITION_SUBTYPE_DATA_SPIFFS: chip_report_printf("SPIFFS"); break;
+            case ESP_PARTITION_SUBTYPE_DATA_LITTLEFS: chip_report_printf("LITTLEFS"); break;
             default: chip_report_printf("0x%02X", partition->subtype); break;
           }
         }

Diff for: docs/en/migration_guides/2.x_to_3.0.rst

@@ -189,9 +189,13 @@ Changes in APIs
 Functional changes
 ******************
 
+* Default pins for some SoCs have been changed to avoid conflicts with other peripherals:
+  * ESP32's UART1 RX and TX pins are now GPIO26 and GPIO27, respectively;
+  * ESP32's UART2 RX and TX pins are now GPIO4 and GPIO25, respectively;
+  * ESP32-S2's UART1 RX and TX pins are now GPIO4 and GPIO5, respectively.
 * It is now possible to detach UART0 pins by calling ``end()`` with no previous call of ``begin()``.
 * It is now possible to call ``setPins()`` before ``begin()`` or in any order.
-* ``setPins(``) will detach any previous pins that have been changed.
+* ``setPins()`` will detach any previous pins that have been changed.
 * ``begin(baud, rx, tx)`` will detach any previous attached pins.
 * ``setPins()`` or ``begin(baud, rx, tx)`` when called at first, will detach console RX0/TX0, attached in boot.
 * Any pin set as -1 in ``begin()`` or ``setPins()`` won't be changed nor detached.
@@ -206,4 +210,4 @@ Functional changes
 
 * In Arduino (and other frameworks) the method named ``flush()`` is intended to send out the transmit buffer content. ``WiFiClient`` and ``WiFiUDP`` method ``flush()`` won't clear the receive buffer anymore. A new method called ``clear()`` is now used for that. Currently ``flush()`` does nothing in ``WiFiClient``, ``WiFiClientSecure`` and ``WiFiUDP``.
 * ``WiFiServer`` has functions ``accept()`` and ``available()`` with the same functionality. In Arduino, ``available()`` should work differently so it is now deprecated.
-* ``WiFiServer`` had unimplemented write functions inherited from ``Print`` class. These are now removed. Also unimplemented method ``stopAll()`` is removed. The methods were unimplemented because WiFiServer doesn't manage connected WiFiClient objects for print-to-all-clients functionality.
+* ``WiFiServer`` had unimplemented write functions inherited from ``Print`` class. These are now removed. Also unimplemented method ``stopAll()`` is removed. The methods were unimplemented because ``WiFiServer`` doesn't manage connected ``WiFiClient`` objects for print-to-all-clients functionality.

Diff for: tests/touch/touch.ino

@@ -25,7 +25,7 @@ uint8_t TOUCH_GPIOS[] = { 4, 2, 15, 13, 12, 14, 27, 33, 32 };
 
 #define NO_TOUCH_GPIO 25
 
-#define RELEASED_VALUE 80  //80+ read value to pass test
+#define RELEASED_VALUE 75  //75+ read value to pass test
 #define PRESSED_VALUE 20   //20- read value to pass test
 #define INTERRUPT_THRESHOLD 40