Skip to content
New issue

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

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

Already on GitHub? # to your account

Don't swallow errors #384

Merged
merged 2 commits into from
Mar 8, 2024
Merged

Conversation

rpdelaney
Copy link
Contributor

  • Change 'pattern does not match any files' warning to fatal
  • Also raise a fatal error if the config file is not valid

Closes #383

@@ -25,7 +25,7 @@ async function run() {
if (config.input_files) {
const patterns = unmatchedPatterns(config.input_files);
patterns.forEach((pattern) =>
console.warn(`🤔 Pattern '${pattern}' does not match any files.`)
throw new Error(`⚠️ Pattern '${pattern}' does not match any files.`)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

makes sense to me. thanks @rpdelaney

@softprops softprops merged commit fe71fd3 into softprops:master Mar 8, 2024
@softprops
Copy link
Owner

this should be available in the new v2 tag

cgrindel-self-hosted-renovate bot referenced this pull request in cgrindel/renovate-bot Mar 9, 2024
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[softprops/action-gh-release](https://github.com/softprops/action-gh-release)
| action | patch | `v2.0.0` -> `v2.0.1` |

---

### Release Notes

<details>
<summary>softprops/action-gh-release
(softprops/action-gh-release)</summary>

###
[`v2.0.1`](https://github.com/softprops/action-gh-release/releases/tag/v2.0.1)

[Compare
Source](https://github.com/softprops/action-gh-release/compare/v2.0.0...v2.0.1)

- Add support for make_latest property
[https://github.com/softprops/action-gh-release/pull/304](https://github.com/softprops/action-gh-release/pull/304)
via [@&#8203;samueljseay](https://github.com/samueljseay)
- Fail run if files setting contains invalid patterns
[https://github.com/softprops/action-gh-release/pull/384](https://github.com/softprops/action-gh-release/pull/384)
via [@&#8203;rpdelaney](https://github.com/rpdelaney)
- Add support for proxy env variables (don't use node-fetch)
[https://github.com/softprops/action-gh-release/pull/386](https://github.com/softprops/action-gh-release/pull/386)/
via [@&#8203;timor-raiman](https://github.com/timor-raiman)
- Suppress confusing warning when input_files is empty
[https://github.com/softprops/action-gh-release/pull/389](https://github.com/softprops/action-gh-release/pull/389)
via [@&#8203;Drowze](https://github.com/Drowze)

</details>

---

### Configuration

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

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the
rebase/retry checkbox.

👻 **Immortal**: This PR will be recreated if closed unmerged. Get
[config help](https://github.com/renovatebot/renovate/discussions) if
that's undesired.

---

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

---

This PR has been generated by [Renovate
Bot](https://github.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi4xMDAuMCIsInVwZGF0ZWRJblZlciI6IjM2LjEwMC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: Self-hosted Renovate Bot <361546+cgrindel-self-hosted-renovate[bot]@users.noreply.github.enterprise.com>
@rpdelaney rpdelaney deleted the rpd-dont_swallow_error branch March 9, 2024 19:24
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The action should fail with an error if the files: settings are not valid
2 participants