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

abort the build on the error #935

Merged
merged 6 commits into from Mar 20, 2023
Merged

Conversation

dsame
Copy link
Contributor

@dsame dsame commented Feb 10, 2023

Description:
Current logic assumes the build is not interrupted on some exception. This leads the build looks succeeded while it is failed in fact.

The PR makeы the build to fail if some of them have happened.

Related issue:
link to the related issue.

Check list:

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

@dsame dsame force-pushed the invalid-creds2 branch 2 times, most recently from 4d1e2fb to ae382cd Compare February 10, 2023 11:44
@dsame dsame marked this pull request as ready for review February 13, 2023 07:36
@dsame dsame requested a review from a team as a code owner February 13, 2023 07:36
@IvanZosimov IvanZosimov self-requested a review February 13, 2023 09:46
src/classes/issues-processor.ts Outdated Show resolved Hide resolved
src/classes/issues-processor.ts Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
@dsame dsame force-pushed the invalid-creds2 branch 8 times, most recently from c584de6 to b1cebe0 Compare February 14, 2023 11:25
package.json Outdated Show resolved Hide resolved
src/classes/issues-processor.ts Outdated Show resolved Hide resolved
@dsame dsame force-pushed the invalid-creds2 branch 2 times, most recently from d1d7035 to 6f4f2e2 Compare February 27, 2023 19:34
@dsame dsame merged commit 01aa532 into actions:main Mar 20, 2023
8 checks passed
kayman-mk pushed a commit to Hapag-Lloyd/Repository-Templates that referenced this pull request Nov 19, 2023
[![Mend Renovate logo
banner](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [actions/stale](https://togithub.com/actions/stale) | action | major |
`v7.0.0` -> `v8.0.0` |

---

### Release Notes

<details>
<summary>actions/stale (actions/stale)</summary>

### [`v8.0.0`](https://togithub.com/actions/stale/releases/tag/v8.0.0)

[Compare
Source](https://togithub.com/actions/stale/compare/v7.0.0...v8.0.0)

:warning: This version contains breaking changes :warning:

#### What's Changed

- New option labels-to-remove-when-stale enables users to specify list
of comma delimited labels that will be removed when the issue or PR
becomes stale by [@&#8203;panticmilos](https://togithub.com/panticmilos)
[actions/stale#770
- Skip deleting the branch in the upstream of a forked repo by
[@&#8203;dsame](https://togithub.com/dsame)
[actions/stale#913
- abort the build on the error by
[@&#8203;dsame](https://togithub.com/dsame) in
[actions/stale#935

#### Breaking Changes

- In this release we prevent scenarios when the build is not interrupted
on some exceptions, which led to successful builds when they are
supposed to fail

#### Example

```yaml
name: 'Remove labels when the issue or PR becomes stale'
on:
  schedule:
    - cron: '30 1 * * *'

permissions:
    pull-requests: write

jobs:
  stale:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/stale@v8
        with:
          labels-to-remove-when-stale: 'label1,label2'
```

</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 is behind base branch, 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/Hapag-Lloyd/Repository-Templates).

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

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
kayman-mk pushed a commit to Hapag-Lloyd/Renovate-Global-Configuration that referenced this pull request Nov 19, 2023
[![Mend Renovate logo
banner](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [actions/stale](https://togithub.com/actions/stale) | action | major |
`v7.0.0` -> `v8.0.0` |

---

### Release Notes

<details>
<summary>actions/stale (actions/stale)</summary>

### [`v8.0.0`](https://togithub.com/actions/stale/releases/tag/v8.0.0)

[Compare
Source](https://togithub.com/actions/stale/compare/v7.0.0...v8.0.0)

:warning: This version contains breaking changes :warning:

#### What's Changed

- New option labels-to-remove-when-stale enables users to specify list
of comma delimited labels that will be removed when the issue or PR
becomes stale by [@&#8203;panticmilos](https://togithub.com/panticmilos)
[actions/stale#770
- Skip deleting the branch in the upstream of a forked repo by
[@&#8203;dsame](https://togithub.com/dsame)
[actions/stale#913
- abort the build on the error by
[@&#8203;dsame](https://togithub.com/dsame) in
[actions/stale#935

#### Breaking Changes

- In this release we prevent scenarios when the build is not interrupted
on some exceptions, which led to successful builds when they are
supposed to fail

#### Example

```yaml
name: 'Remove labels when the issue or PR becomes stale'
on:
  schedule:
    - cron: '30 1 * * *'

permissions:
    pull-requests: write

jobs:
  stale:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/stale@v8
        with:
          labels-to-remove-when-stale: 'label1,label2'
```

</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 is behind base branch, 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/Hapag-Lloyd/Renovate-Global-Configuration).

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

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants