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.

Sign up for GitHub

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

Jump to bottom

Introduce 'paths_filter' option and 'changed_files' output #181

Merged

fkirc merged 9 commits into fkirc:master from paescuj:paths

Jan 17, 2022

Collaborator

paescuj commented Jan 16, 2022 •

edited

Loading

See #159

Submitted as one PR since 'paths_filter' option and 'changed_files' output are closely connected. Hope that's okay for you, @fkirc!

Update README

Closes #139

paescuj added 5 commits

January 16, 2022 16:31


          Udpdate all dependencies

c85f9c0

See fkirc#159


          Switch to Node.js v16

758413e

Node.js v16 is current LTS and now supported as action runtime


          Only format files in 'src' folder

577e2f4


          Introduce 'reason' output

c91dd98

See fkirc#159


          Introduce 'paths_filter' option and 'changed_files' output

4f0293b

See fkirc#159

paescuj marked this pull request as draft

January 16, 2022 18:32

paescuj added 4 commits

January 16, 2022 21:53


          Fix action.yml and first part of README update

803a88b


          Merge branch 'master' into paths

4a5a2a8


          Rebuild

93e4f4e


          Update README.md to reflect new input/outputs

ceeb5b9

paescuj marked this pull request as ready for review

January 17, 2022 11:28

Collaborator Author

paescuj commented Jan 17, 2022

Ready from my side 😃

Owner

fkirc commented Jan 17, 2022

The code looks good to me, let`s put it into master for community-testing 👍

fkirc merged commit 4a0432e into fkirc:master

Collaborator Author

paescuj commented Jan 17, 2022

The code looks good to me, let`s put it into master for community-testing 👍

Nice, thanks 👍

paescuj deleted the paths branch

January 17, 2022 19:14

maximilianbraun referenced this pull request in crossplane-contrib/xp-testing


          chore(deps): update fkirc/skip-duplicate-actions action to v5 (#16)

78be738

[![Mend Renovate logo
banner](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[fkirc/skip-duplicate-actions](https://github.com/fkirc/skip-duplicate-actions)
| action | major | `v2.2.0` -> `v5.3.1` |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>fkirc/skip-duplicate-actions
(fkirc/skip-duplicate-actions)</summary>

###
[`v5.3.1`](https://github.com/fkirc/skip-duplicate-actions/compare/v5.3.0...v5.3.1)

[Compare
Source](https://github.com/fkirc/skip-duplicate-actions/compare/v5.3.0...v5.3.1)

###
[`v5.3.0`](https://github.com/fkirc/skip-duplicate-actions/releases/tag/v5.3.0)

[Compare
Source](https://github.com/fkirc/skip-duplicate-actions/compare/v5.2.0...v5.3.0)

- Retry API requests on server errors by
[@&#8203;paescuj](https://github.com/paescuj) in
[https://github.com/fkirc/skip-duplicate-actions/pull/288](https://github.com/fkirc/skip-duplicate-actions/pull/288)
-   Update dependencies

###
[`v5.2.0`](https://github.com/fkirc/skip-duplicate-actions/releases/tag/v5.2.0)

[Compare
Source](https://github.com/fkirc/skip-duplicate-actions/compare/v5.1.0...v5.2.0)

- Add a job summary by [@&#8203;paescuj](https://github.com/paescuj)
in
[https://github.com/fkirc/skip-duplicate-actions/pull/277](https://github.com/fkirc/skip-duplicate-actions/pull/277)
- Cleanup README by [@&#8203;fkirc](https://github.com/fkirc) in
[https://github.com/fkirc/skip-duplicate-actions/pull/278](https://github.com/fkirc/skip-duplicate-actions/pull/278)
- Add graphical representation of backtracking-algorithm by
[@&#8203;paescuj](https://github.com/paescuj) in
[https://github.com/fkirc/skip-duplicate-actions/pull/279](https://github.com/fkirc/skip-duplicate-actions/pull/279)
- Add FAQ section with solution for required matrix jobs by
[@&#8203;paescuj](https://github.com/paescuj) in
[https://github.com/fkirc/skip-duplicate-actions/pull/283](https://github.com/fkirc/skip-duplicate-actions/pull/283)
- Update dependencies & rebuild by
[@&#8203;paescuj](https://github.com/paescuj) in
[https://github.com/fkirc/skip-duplicate-actions/pull/285](https://github.com/fkirc/skip-duplicate-actions/pull/285)

###
[`v5.1.0`](https://github.com/fkirc/skip-duplicate-actions/releases/tag/v5.1.0)

[Compare
Source](https://github.com/fkirc/skip-duplicate-actions/compare/v5.0.0...v5.1.0)

Allow `release` for `do_not_skip` input
([https://github.com/fkirc/skip-duplicate-actions/pull/273](https://github.com/fkirc/skip-duplicate-actions/pull/273))

###
[`v5.0.0`](https://github.com/fkirc/skip-duplicate-actions/releases/tag/v5.0.0)

[Compare
Source](https://github.com/fkirc/skip-duplicate-actions/compare/v4.0.0...v5.0.0)

Various optimizations:

-   Check all inputs *before* doing any API requests
-   Stricter check for `doNotSkip` input
-   Exclude current workflow run from list of all workflow runs
-   Get rid of redundant and unnecessary lines
-   Add some explaining comments
-   Fix bug with empty `head_repository`
-   Action *might* be a bit faster 🚀

**Breaking changes:**

- Previously, on input and API request errors the action might have
exited successfully but with outputs `should_skip = false` and `reason =
no_workflow_information`. Now, the action will exit with an error
instead.

To make sure that subsequent jobs will not be skipped in such cases it's
recommended to set `continue-on-error: true` in the
`skip-duplicate-actions` job.
Note: This has already been recommended in the past regardless of this
change.
- The structure of the `skipped_by` output (also in `paths_filter`) has
slightly changed for the sake of consistency and alignment with GitHub
API:
    -   `runId` is now `id`
    -   `html_url` is now `htmlUrl`
    -   Full example:
        ```json
        {
         "id": 1709469369,
         "runNumber": 737,
         "event": "pull_request",
         "treeHash": "e3434bb7aeb3047d7df948f09419ac96cf03d73e",
         "commitHash": "4a0432e823468ecff81a978165cb35586544c795",
         "status": "completed",
         "conclusion": "success",
"htmlUrl":
"https://github.com/fkirc/skip-duplicate-actions/actions/runs/1709469369",
         "branch": "master",
         "repo": "fkirc/skip-duplicate-actions",
         "workflowId": 2640563,
         "createdAt": "2022-01-17T18:56:06Z"
        }
        ```

Other changes:

-   Update dependencies

**Full Changelog**:
fkirc/skip-duplicate-actions@v4...v5.0.0

###
[`v4.0.0`](https://github.com/fkirc/skip-duplicate-actions/releases/tag/v4.0.0)

[Compare
Source](https://github.com/fkirc/skip-duplicate-actions/compare/v3.4.1...v4.0.0)

Advanced path-filtering and output options
([https://github.com/fkirc/skip-duplicate-actions/pull/181](https://github.com/fkirc/skip-duplicate-actions/pull/181))

###
[`v3.4.1`](https://github.com/fkirc/skip-duplicate-actions/releases/tag/v3.4.1)

[Compare
Source](https://github.com/fkirc/skip-duplicate-actions/compare/v3.4.0...v3.4.1)

Fix same_content_new bug (runs would be inappropriately skipped)
([https://github.com/fkirc/skip-duplicate-actions/pull/125](https://github.com/fkirc/skip-duplicate-actions/pull/125))

###
[`v3.4.0`](https://github.com/fkirc/skip-duplicate-actions/releases/tag/v3.4.0)

[Compare
Source](https://github.com/fkirc/skip-duplicate-actions/compare/v3.3.0...v3.4.0)

Disable cancellation by default
([#&#8203;93](https://github.com/fkirc/skip-duplicate-actions/issues/93))

###
[`v3.3.0`](https://github.com/fkirc/skip-duplicate-actions/releases/tag/v3.3.0)

[Compare
Source](https://github.com/fkirc/skip-duplicate-actions/compare/v3.2.0...v3.3.0)

Improve logging of paths-skipping
([#&#8203;80](https://github.com/fkirc/skip-duplicate-actions/issues/80))

###
[`v3.2.0`](https://github.com/fkirc/skip-duplicate-actions/releases/tag/v3.2.0)

[Compare
Source](https://github.com/fkirc/skip-duplicate-actions/compare/v3.1.0...v3.2.0)

Optional duplicate skipping
([#&#8203;74](https://github.com/fkirc/skip-duplicate-actions/issues/74))

###
[`v3.1.0`](https://github.com/fkirc/skip-duplicate-actions/releases/tag/v3.1.0)

[Compare
Source](https://github.com/fkirc/skip-duplicate-actions/compare/v3.0.0...v3.1.0)

Add an option to skip outdated runs
([https://github.com/fkirc/skip-duplicate-actions/pull/70](https://github.com/fkirc/skip-duplicate-actions/pull/70))

###
[`v3.0.0`](https://github.com/fkirc/skip-duplicate-actions/releases/tag/v3.0.0)

[Compare
Source](https://github.com/fkirc/skip-duplicate-actions/compare/v2.2.0...v3.0.0)

Make concurrent skipping more powerful and easier to understand
([https://github.com/fkirc/skip-duplicate-actions/pull/66](https://github.com/fkirc/skip-duplicate-actions/pull/66))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/crossplane-contrib/xp-testing).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40Ni4wIiwidXBkYXRlZEluVmVyIjoiMzcuNTkuOCIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet