Skip to content

fix: \\n should be changed to \n only when escapeSpecialCharacters is set#91

Merged
JoshuaKGoldberg merged 2 commits intodmnd:mainfrom
fuenfundachtzig:main
Apr 10, 2024
Merged

fix: \\n should be changed to \n only when escapeSpecialCharacters is set#91
JoshuaKGoldberg merged 2 commits intodmnd:mainfrom
fuenfundachtzig:main

Conversation

@fuenfundachtzig
Copy link
Copy Markdown
Contributor

@fuenfundachtzig fuenfundachtzig commented Mar 31, 2024

This PR proposes a fix to the issue #90.

PR Checklist

@JoshuaKGoldberg JoshuaKGoldberg changed the title \\n should be changed to \n only when escapeSpecialCharacters is set fix: \\n should be changed to \n only when escapeSpecialCharacters is set Apr 9, 2024
Copy link
Copy Markdown
Collaborator

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

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

LGTM! Could you add a test to dedent-tests.ts please?

@JoshuaKGoldberg JoshuaKGoldberg added the status: waiting for author Needs an action taken by the original poster label Apr 9, 2024
@fuenfundachtzig
Copy link
Copy Markdown
Contributor Author

LGTM! Could you add a test to dedent-tests.ts please?

Thanks! I have added a test.

Copy link
Copy Markdown
Collaborator

@JoshuaKGoldberg JoshuaKGoldberg left a comment

Choose a reason for hiding this comment

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

🔥 thanks!

@JoshuaKGoldberg JoshuaKGoldberg removed the status: waiting for author Needs an action taken by the original poster label Apr 10, 2024
@JoshuaKGoldberg JoshuaKGoldberg merged commit 4c133c1 into dmnd:main Apr 10, 2024
@github-actions
Copy link
Copy Markdown

🎉 This is included in version v1.5.2 🎉

The release is available on:

Cheers! 📦🚀

renovate bot referenced this pull request in JoshuaKGoldberg/eslint-plugin-expect-type Apr 13, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [dedent](https://togithub.com/dmnd/dedent) | [`1.5.1` ->
`1.5.2`](https://renovatebot.com/diffs/npm/dedent/1.5.1/1.5.2) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/dedent/1.5.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/dedent/1.5.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/dedent/1.5.1/1.5.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/dedent/1.5.1/1.5.2?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>dmnd/dedent (dedent)</summary>

### [`v1.5.2`](https://togithub.com/dmnd/dedent/releases/tag/v1.5.2)

[Compare
Source](https://togithub.com/dmnd/dedent/compare/v1.5.1...v1.5.2)

#### What's Changed

- chore: remove files related to Flow by
[@&#8203;yinm](https://togithub.com/yinm) in
[https://github.com/dmnd/dedent/pull/77](https://togithub.com/dmnd/dedent/pull/77)
- build: onboarded to create-typescript-app (mostly) by
[@&#8203;JoshuaKGoldberg](https://togithub.com/JoshuaKGoldberg) in
[https://github.com/dmnd/dedent/pull/55](https://togithub.com/dmnd/dedent/pull/55)
- chore: fix special characters escaping issue link in README by
[@&#8203;auvred](https://togithub.com/auvred) in
[https://github.com/dmnd/dedent/pull/82](https://togithub.com/dmnd/dedent/pull/82)
- chore: replace last yarn references with pnpm by
[@&#8203;JoshuaKGoldberg](https://togithub.com/JoshuaKGoldberg) in
[https://github.com/dmnd/dedent/pull/86](https://togithub.com/dmnd/dedent/pull/86)
- docs: add explicit install command to README.md by
[@&#8203;JoshuaKGoldberg](https://togithub.com/JoshuaKGoldberg) in
[https://github.com/dmnd/dedent/pull/84](https://togithub.com/dmnd/dedent/pull/84)
- fix: switch babel-plugin-macros to a peer dependency (again) by
[@&#8203;JoshuaKGoldberg](https://togithub.com/JoshuaKGoldberg) in
[https://github.com/dmnd/dedent/pull/89](https://togithub.com/dmnd/dedent/pull/89)
- build(deps-dev): bump
[@&#8203;babel/traverse](https://togithub.com/babel/traverse) from
7.22.8 to 7.23.6 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/dmnd/dedent/pull/87](https://togithub.com/dmnd/dedent/pull/87)
- fix: \n should be changed to \n only when escapeSpecialCharacters is
set by [@&#8203;fuenfundachtzig](https://togithub.com/fuenfundachtzig)
in
[https://github.com/dmnd/dedent/pull/91](https://togithub.com/dmnd/dedent/pull/91)

#### New Contributors

- [@&#8203;yinm](https://togithub.com/yinm) made their first
contribution in
[https://github.com/dmnd/dedent/pull/77](https://togithub.com/dmnd/dedent/pull/77)
- [@&#8203;auvred](https://togithub.com/auvred) made their first
contribution in
[https://github.com/dmnd/dedent/pull/82](https://togithub.com/dmnd/dedent/pull/82)
- [@&#8203;dependabot](https://togithub.com/dependabot) made their first
contribution in
[https://github.com/dmnd/dedent/pull/87](https://togithub.com/dmnd/dedent/pull/87)
- [@&#8203;fuenfundachtzig](https://togithub.com/fuenfundachtzig) made
their first contribution in
[https://github.com/dmnd/dedent/pull/91](https://togithub.com/dmnd/dedent/pull/91)

**Full Changelog**:
dmnd/dedent@v1.5.1...v1.5.2

</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 [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/JoshuaKGoldberg/eslint-plugin-expect-type).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yNjkuMiIsInVwZGF0ZWRJblZlciI6IjM3LjI2OS4yIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
jcfranco referenced this pull request in Esri/calcite-design-system Apr 16, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [dedent](https://togithub.com/dmnd/dedent) | [`1.5.1` ->
`1.5.3`](https://renovatebot.com/diffs/npm/dedent/1.5.1/1.5.3) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/dedent/1.5.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/dedent/1.5.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/dedent/1.5.1/1.5.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/dedent/1.5.1/1.5.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>dmnd/dedent (dedent)</summary>

### [`v1.5.3`](https://togithub.com/dmnd/dedent/releases/tag/v1.5.3)

[Compare
Source](https://togithub.com/dmnd/dedent/compare/v1.5.2...v1.5.3)

#### What's Changed

- fix: remove engines and provenance requirements by
[@&#8203;JoshuaKGoldberg](https://togithub.com/JoshuaKGoldberg) in
[https://github.com/dmnd/dedent/pull/93](https://togithub.com/dmnd/dedent/pull/93)

**Full Changelog**:
dmnd/dedent@v1.5.2...v1.5.3

### [`v1.5.2`](https://togithub.com/dmnd/dedent/releases/tag/v1.5.2)

[Compare
Source](https://togithub.com/dmnd/dedent/compare/v1.5.1...v1.5.2)

#### What's Changed

- chore: remove files related to Flow by
[@&#8203;yinm](https://togithub.com/yinm) in
[https://github.com/dmnd/dedent/pull/77](https://togithub.com/dmnd/dedent/pull/77)
- build: onboarded to create-typescript-app (mostly) by
[@&#8203;JoshuaKGoldberg](https://togithub.com/JoshuaKGoldberg) in
[https://github.com/dmnd/dedent/pull/55](https://togithub.com/dmnd/dedent/pull/55)
- chore: fix special characters escaping issue link in README by
[@&#8203;auvred](https://togithub.com/auvred) in
[https://github.com/dmnd/dedent/pull/82](https://togithub.com/dmnd/dedent/pull/82)
- chore: replace last yarn references with pnpm by
[@&#8203;JoshuaKGoldberg](https://togithub.com/JoshuaKGoldberg) in
[https://github.com/dmnd/dedent/pull/86](https://togithub.com/dmnd/dedent/pull/86)
- docs: add explicit install command to README.md by
[@&#8203;JoshuaKGoldberg](https://togithub.com/JoshuaKGoldberg) in
[https://github.com/dmnd/dedent/pull/84](https://togithub.com/dmnd/dedent/pull/84)
- fix: switch babel-plugin-macros to a peer dependency (again) by
[@&#8203;JoshuaKGoldberg](https://togithub.com/JoshuaKGoldberg) in
[https://github.com/dmnd/dedent/pull/89](https://togithub.com/dmnd/dedent/pull/89)
- build(deps-dev): bump
[@&#8203;babel/traverse](https://togithub.com/babel/traverse) from
7.22.8 to 7.23.6 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/dmnd/dedent/pull/87](https://togithub.com/dmnd/dedent/pull/87)
- fix: \n should be changed to \n only when escapeSpecialCharacters is
set by [@&#8203;fuenfundachtzig](https://togithub.com/fuenfundachtzig)
in
[https://github.com/dmnd/dedent/pull/91](https://togithub.com/dmnd/dedent/pull/91)

#### New Contributors

- [@&#8203;yinm](https://togithub.com/yinm) made their first
contribution in
[https://github.com/dmnd/dedent/pull/77](https://togithub.com/dmnd/dedent/pull/77)
- [@&#8203;auvred](https://togithub.com/auvred) made their first
contribution in
[https://github.com/dmnd/dedent/pull/82](https://togithub.com/dmnd/dedent/pull/82)
- [@&#8203;dependabot](https://togithub.com/dependabot) made their first
contribution in
[https://github.com/dmnd/dedent/pull/87](https://togithub.com/dmnd/dedent/pull/87)
- [@&#8203;fuenfundachtzig](https://togithub.com/fuenfundachtzig) made
their first contribution in
[https://github.com/dmnd/dedent/pull/91](https://togithub.com/dmnd/dedent/pull/91)

**Full Changelog**:
dmnd/dedent@v1.5.1...v1.5.2

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "before 5am every weekday" in timezone
America/Los_Angeles, Automerge - 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, 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/Esri/calcite-design-system).

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

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.

🐛 Bug: \\n -> \n should only happen with template strings

2 participants