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

CLI: Don't use logger to print information #12477

Merged
merged 12 commits into from
Mar 21, 2022
Merged

Conversation

fisker
Copy link
Member

@fisker fisker commented Mar 16, 2022

Description

Should not use logger.log to print information, for example

prettier -v --loglevel silent

Should still print the version.

Checklist

  • I’ve added tests to confirm my change works.
  • (If changing the API or CLI) I’ve documented the changes I’ve made (in the docs/ directory).
  • (If the change is user-facing) I’ve added my changes to changelog_unreleased/*/XXXX.md file following changelog_unreleased/TEMPLATE.md.
  • I’ve read the contributing guidelines.

Try the playground for this PR

@fisker fisker marked this pull request as ready for review March 16, 2022 10:40
Copy link
Member

@sosukesuzuki sosukesuzuki left a comment

Choose a reason for hiding this comment

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

What do you think about to add logger.info(message) method instead of using console.log directly?

@fisker
Copy link
Member Author

fisker commented Mar 17, 2022

I don't think this information is related to logger, it's just a print().

@fisker
Copy link
Member Author

fisker commented Mar 20, 2022

Maybe define a function printToScreen()? @sosukesuzuki

@sosukesuzuki
Copy link
Member

sosukesuzuki commented Mar 20, 2022

Thanks. I prefer that, but I have no strong opinion

@fisker fisker merged commit 5ef8212 into prettier:main Mar 21, 2022
@fisker fisker deleted the loglevel branch March 21, 2022 02:38
crapStone pushed a commit to Calciumdibromid/CaBr2 that referenced this pull request Mar 27, 2022
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [prettier](https://prettier.io) ([source](https://github.com/prettier/prettier)) | devDependencies | patch | [`2.6.0` -> `2.6.1`](https://renovatebot.com/diffs/npm/prettier/2.6.0/2.6.1) |

---

### Release Notes

<details>
<summary>prettier/prettier</summary>

### [`v2.6.1`](https://github.com/prettier/prettier/blob/HEAD/CHANGELOG.md#&#8203;261)

[Compare Source](prettier/prettier@2.6.0...2.6.1)

[diff](prettier/prettier@2.6.0...2.6.1)

##### Ignore `loglevel` when printing information ([#&#8203;12477](prettier/prettier#12477) by [@&#8203;fisker](https://github.com/fisker))

<!-- prettier-ignore -->

```bash
```

</details>

---

### Configuration

📅 **Schedule**: 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, click this checkbox.

---

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

Co-authored-by: cabr2-bot <[email protected]>
Reviewed-on: https://codeberg.org/Calciumdibromid/CaBr2/pulls/1244
Reviewed-by: 6543 <[email protected]>
Co-authored-by: Calciumdibromid Bot <[email protected]>
Co-committed-by: Calciumdibromid Bot <[email protected]>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 22, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants