Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

workflows/publish.yml: Run the workflow on success and failure only. #6531

Merged
merged 4 commits into from
Apr 21, 2022

Conversation

Ouss4
Copy link
Contributor

@Ouss4 Ouss4 commented Apr 4, 2022

Summary

  • This prevents trying to run when the trigger was cancelled or skipped.
    In these cases there will be no event file to upload and the workflow will fail.

  • Update the runners' tags. One of the available runners doesn't have an S2 board anymore.

Impact

Publish workflow.

Related links

https://github.com/Ouss4/arduino-esp32/actions/runs/2083991236
https://github.com/Ouss4/arduino-esp32/actions/runs/2087421730

This prevents trying to run when the trigger was cancelled or skipped.
In these cases there will be no event file to upload.

Signed-off-by: Abdelatif Guettouche <abdelatif.guettouche@espressif.com>
@github-actions
Copy link
Contributor

github-actions bot commented Apr 4, 2022

Unit Test Results

  8 files    8 suites   58s ⏱️
  3 tests   3 ✔️ 0 💤 0
12 runs  12 ✔️ 0 💤 0

Results for commit 5e5f6e4.

♻️ This comment has been updated with latest results.

the part that retrieves the arguments.

Signed-off-by: Abdelatif Guettouche <abdelatif.guettouche@espressif.com>
@Ouss4 Ouss4 added hil_test Run Hardware Tests and removed hil_test Run Hardware Tests labels Apr 4, 2022
Signed-off-by: Abdelatif Guettouche <abdelatif.guettouche@espressif.com>
useful for debugging.

Signed-off-by: Abdelatif Guettouche <abdelatif.guettouche@espressif.com>
@Ouss4 Ouss4 requested a review from me-no-dev April 14, 2022 13:29
@me-no-dev me-no-dev merged commit 323bbbf into espressif:master Apr 21, 2022
@Ouss4 Ouss4 deleted the ci branch April 21, 2022 15:11
Jason2866 added a commit to tasmota/arduino-esp32 that referenced this pull request Apr 21, 2022
* Delete stale.yml

* IDF release/v4.4 b8050b365e (espressif#6594)

* Update camera example to support face detection and recognition (espressif#6603)

Fixes: espressif#6508

* Change pinMode OUTPUT to INPUT_OUTPUT (espressif#6602)

* Change OUTPUT to INPUT_OUTPUT

To match the official Arduino API.

* Changed type of LEDC frequency from double to uint32_t (espressif#6570)

* BUGFIX: FS read + speed improvements for SD (espressif#6569)

* Revert "Edited VFSFileImpl::read to use both read/fread (espressif#6456)"

This reverts commit 7b89b39.

* Added default file buffer size + function to change it by user

* workflows/publish.yml: Run the workflow on success and failure only. (espressif#6531)

* workflows/publish.yml: Run the workflow on success and failure only.
This prevents trying to run when the trigger was cancelled or skipped.
In these cases there will be no event file to upload.

Signed-off-by: Abdelatif Guettouche <abdelatif.guettouche@espressif.com>

* scripts/sketch_utils.sh: Move the logic that gets the build dir after
the part that retrieves the arguments.

Signed-off-by: Abdelatif Guettouche <abdelatif.guettouche@espressif.com>

* workflows/hil.yml: Update the HIL runners tags.

Signed-off-by: Abdelatif Guettouche <abdelatif.guettouche@espressif.com>

* workflows/hil.yml: Remove the Check Artifacts step.  That was only
useful for debugging.

Signed-off-by: Abdelatif Guettouche <abdelatif.guettouche@espressif.com>

* Pull request/Issue Templates and Readme update (espressif#6577)

* Templates and readme

* Templates_and_readme

* Delete libraries/RainMaker directory

* Delete tools/sdk/esp32/include/esp_rainmaker directory

* Delete tools/sdk/esp32s2/include/esp_rainmaker directory

* Delete tools/sdk/esp32s3/include/esp_rainmaker directory

* Delete tools/sdk/esp32c3/include/esp_rainmaker directory

Co-authored-by: Me No Dev <me-no-dev@users.noreply.github.com>
Co-authored-by: Jan Procházka <90197375+P-R-O-C-H-Y@users.noreply.github.com>
Co-authored-by: Abdelatif Guettouche <abdelatif.guettouche@espressif.com>
Co-authored-by: Vojtěch Bartoška <76958047+VojtechBartoska@users.noreply.github.com>
Jason2866 added a commit to tasmota/arduino-esp32 that referenced this pull request Apr 24, 2022
* Delete stale.yml

* IDF release/v4.4 b8050b365e (espressif#6594)

* Update camera example to support face detection and recognition (espressif#6603)

Fixes: espressif#6508

* Change pinMode OUTPUT to INPUT_OUTPUT (espressif#6602)

* Change OUTPUT to INPUT_OUTPUT

To match the official Arduino API.

* Changed type of LEDC frequency from double to uint32_t (espressif#6570)

* BUGFIX: FS read + speed improvements for SD (espressif#6569)

* Revert "Edited VFSFileImpl::read to use both read/fread (espressif#6456)"

This reverts commit 7b89b39.

* Added default file buffer size + function to change it by user

* workflows/publish.yml: Run the workflow on success and failure only. (espressif#6531)

* workflows/publish.yml: Run the workflow on success and failure only.
This prevents trying to run when the trigger was cancelled or skipped.
In these cases there will be no event file to upload.

Signed-off-by: Abdelatif Guettouche <abdelatif.guettouche@espressif.com>

* scripts/sketch_utils.sh: Move the logic that gets the build dir after
the part that retrieves the arguments.

Signed-off-by: Abdelatif Guettouche <abdelatif.guettouche@espressif.com>

* workflows/hil.yml: Update the HIL runners tags.

Signed-off-by: Abdelatif Guettouche <abdelatif.guettouche@espressif.com>

* workflows/hil.yml: Remove the Check Artifacts step.  That was only
useful for debugging.

Signed-off-by: Abdelatif Guettouche <abdelatif.guettouche@espressif.com>

* Pull request/Issue Templates and Readme update (espressif#6577)

Co-authored-by: Me No Dev <me-no-dev@users.noreply.github.com>
Co-authored-by: Jan Procházka <90197375+P-R-O-C-H-Y@users.noreply.github.com>
Co-authored-by: Abdelatif Guettouche <abdelatif.guettouche@espressif.com>
Co-authored-by: Vojtěch Bartoška <76958047+VojtechBartoska@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hil_test Run Hardware Tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants