Skip to content

Conversation

sosukesuzuki
Copy link
Member

Description

#16850

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

Copy link

github-actions bot commented Nov 24, 2024

Size Change: 0 B

Total Size: 7.86 MB

ℹ️ View Unchanged
Filename Size
./dist/bin 4.1 kB
./dist/bin/prettier.cjs 2.24 kB
./dist/doc.d.ts 7.42 kB
./dist/doc.js 53.1 kB
./dist/doc.mjs 49.6 kB
./dist/index.cjs 36.8 kB
./dist/index.d.ts 26.8 kB
./dist/index.mjs 738 kB
./dist/internal 4.1 kB
./dist/internal/cli.mjs 125 kB
./dist/LICENSE 206 kB
./dist/package.json 6.3 kB
./dist/plugins 4.1 kB
./dist/plugins/acorn.d.ts 109 B
./dist/plugins/acorn.js 152 kB
./dist/plugins/acorn.mjs 152 kB
./dist/plugins/angular.d.ts 177 B
./dist/plugins/angular.js 85.8 kB
./dist/plugins/angular.mjs 85.1 kB
./dist/plugins/babel.d.ts 419 B
./dist/plugins/babel.js 314 kB
./dist/plugins/babel.mjs 314 kB
./dist/plugins/estree.d.ts 11 B
./dist/plugins/estree.js 201 kB
./dist/plugins/estree.mjs 200 kB
./dist/plugins/flow.d.ts 90 B
./dist/plugins/flow.js 679 kB
./dist/plugins/flow.mjs 679 kB
./dist/plugins/glimmer.d.ts 93 B
./dist/plugins/glimmer.js 137 kB
./dist/plugins/glimmer.mjs 137 kB
./dist/plugins/graphql.d.ts 93 B
./dist/plugins/graphql.js 43.6 kB
./dist/plugins/graphql.mjs 43 kB
./dist/plugins/html.d.ts 139 B
./dist/plugins/html.js 151 kB
./dist/plugins/html.mjs 151 kB
./dist/plugins/markdown.d.ts 127 B
./dist/plugins/markdown.js 149 kB
./dist/plugins/markdown.mjs 148 kB
./dist/plugins/meriyah.d.ts 93 B
./dist/plugins/meriyah.js 131 kB
./dist/plugins/meriyah.mjs 131 kB
./dist/plugins/postcss.d.ts 121 B
./dist/plugins/postcss.js 153 kB
./dist/plugins/postcss.mjs 153 kB
./dist/plugins/typescript.d.ts 96 B
./dist/plugins/typescript.js 896 kB
./dist/plugins/typescript.mjs 895 kB
./dist/plugins/yaml.d.ts 90 B
./dist/plugins/yaml.js 122 kB
./dist/plugins/yaml.mjs 122 kB
./dist/README.md 4.03 kB
./dist/standalone.d.ts 1.37 kB
./dist/standalone.js 82.1 kB
./dist/standalone.mjs 81.8 kB

compressed-size-action

@sosukesuzuki sosukesuzuki mentioned this pull request Nov 24, 2024
11 tasks
@sosukesuzuki sosukesuzuki self-assigned this Nov 24, 2024
Copy link
Member

@kachkaev kachkaev left a comment

Choose a reason for hiding this comment

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

Nice! Left a couple of small suggestions.

Blog post preview:
https://deploy-preview-16851--prettier.netlify.app/blog/2024/11/25/3.4.0

---
author: "Sosuke Suzuki (@sosukesuzuki)"
authorURL: "https://github.com/sosukesuzuki"
title: "Prettier 3.4: Fixes a lot of bugs"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
title: "Prettier 3.4: Fixes a lot of bugs"
title: "Prettier 3.4: A lot of bug fixes"

Optional: I could not find a blog post title starting with a verb, but there was https://prettier.io/blog/2019/06/06/1.18.0 for example

// Input
const string = `${[[1, 2], [3, 4]]}`

// Prettier 3.2
Copy link
Member

Choose a reason for hiding this comment

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

I believe this should be

Suggested change
// Prettier 3.2
// Prettier 3.3

here and below

}
```

#### Fixed wrong trailing comma position after comment in SCSS ([#16617](https://github.com/prettier/prettier/pull/16617) by [@Ma-hawaj](https://github.com/Ma-hawaj), [@fisker](https://github.com/fisker))
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
#### Fixed wrong trailing comma position after comment in SCSS ([#16617](https://github.com/prettier/prettier/pull/16617) by [@Ma-hawaj](https://github.com/Ma-hawaj), [@fisker](https://github.com/fisker))
#### Fix wrong trailing comma position after comment in SCSS ([#16617](https://github.com/prettier/prettier/pull/16617) by [@Ma-hawaj](https://github.com/Ma-hawaj), [@fisker](https://github.com/fisker))


### Markdown

#### Removed excessive spaces after line prefixes for unordered lists in Markdown ([#15526](https://github.com/prettier/prettier/pull/15526) by [@TomasLudvik](https://github.com/TomasLudvik))
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
#### Removed excessive spaces after line prefixes for unordered lists in Markdown ([#15526](https://github.com/prettier/prettier/pull/15526) by [@TomasLudvik](https://github.com/TomasLudvik))
#### Remove excessive spaces after line prefixes for unordered lists in Markdown ([#15526](https://github.com/prettier/prettier/pull/15526) by [@TomasLudvik](https://github.com/TomasLudvik))

@fisker
Copy link
Member

fisker commented Nov 25, 2024

@sosukesuzuki Please wait for me to fix #16860

Copy link
Member

Choose a reason for hiding this comment

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

N.B. We’ll need to change the date if we don’t manage to release 3.4 by the end of today AoE

@sosukesuzuki sosukesuzuki merged commit 3542f13 into main Nov 26, 2024
26 checks passed
@sosukesuzuki sosukesuzuki deleted the 3.4-blog-post branch November 26, 2024 05:02
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.

3 participants