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

Add --no-config option #250

Merged
merged 1 commit into from
Feb 16, 2024
Merged

Conversation

jasonmccreary
Copy link
Contributor

@jasonmccreary jasonmccreary commented Feb 16, 2024

Currently, pint will merge any project config with the --preset option. This was slightly unexpected and makes it difficult to quickly use one of the presents from within a project that has a custom pint.json config.

You can get around this by passing an invalid path to --config. But that's a bit hacky. I felt a --no-config option was nice and provides an explicit way to prevent pint from merging any project config.

For example, to apply the laravel preset to a set of stub files:

pint --no-config --preset laravel -- stubs

This is indeed a real world use case where a project has it's own rules configured within pint.json, but contains stub files which are formatted to the laravel preset.

@jasonmccreary jasonmccreary changed the title Add --no-config-option Add --no-config option Feb 16, 2024
@taylorotwell taylorotwell merged commit 3ca3831 into laravel:main Feb 16, 2024
6 checks passed
itinerare pushed a commit to itinerare/Alcyone that referenced this pull request Feb 26, 2024
This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [laravel/pint](https://laravel.com) ([source](https://github.com/laravel/pint)) | `1.13.11` -> `1.14.0` | [![age](https://developer.mend.io/api/mc/badges/age/packagist/laravel%2fpint/1.14.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/packagist/laravel%2fpint/1.14.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/packagist/laravel%2fpint/1.13.11/1.14.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/packagist/laravel%2fpint/1.13.11/1.14.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) |

---

### Release Notes

<details>
<summary>laravel/pint (laravel/pint)</summary>

### [`v1.14.0`](https://github.com/laravel/pint/blob/HEAD/CHANGELOG.md#v1140---2024-02-20)

[Compare Source](laravel/pint@v1.13.11...v1.14.0)

-   Add `--no-config` option by [@&#8203;jasonmccreary](https://github.com/jasonmccreary) in laravel/pint#250

</details>

---

### Configuration

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

🚦 **Automerge**: Enabled.

♻ **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 [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yMTQuMCIsInVwZGF0ZWRJblZlciI6IjM3LjIxNC4wIiwidGFyZ2V0QnJhbmNoIjoiZGV2ZWxvcCJ9-->

Reviewed-on: https://code.itinerare.net/itinerare/Alcyone/pulls/6
Co-authored-by: Amadeus[bot] <amadeus@itinerare.net>
Co-committed-by: Amadeus[bot] <amadeus@itinerare.net>
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.

2 participants