fastify / fastify/github-action-merge-dependabot

Change the default value for `target`

Open
#128 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
185
Forks
41
Avg merge
6h 6m
Merged PRs (30d)
4

Description

### Prerequisites

- [X] I have written a descriptive issue title
- [X] I have searched existing issues to ensure the feature has not already been requested

### 🚀 Feature Proposal

This proposal aims to make the API simpler to understand; it doesn't change the actual behaviour, in fact it wants to keep it.

The only difference is that instead of using the special value `any` I’m proposing to just use an _empty_ value (`''`, `undefined`…) to achieve same thing.

### Motivation

- we don't need to create a special value to disable the `target` check, we just leave it empty
- we simplify the validation of `target` so it would only accept valid SemVer identifiers

### Example

N/A

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.