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

2.6 blog post #12412

Merged
merged 26 commits into from
Mar 16, 2022
Merged

2.6 blog post #12412

merged 26 commits into from
Mar 16, 2022

Conversation

sosukesuzuki
Copy link
Member

@sosukesuzuki sosukesuzuki commented Mar 7, 2022

Description

https://deploy-preview-12412--prettier.netlify.app/blog/2022/03/15/2.6.0.html

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

@sosukesuzuki sosukesuzuki mentioned this pull request Mar 7, 2022
11 tasks
@kachkaev
Copy link
Member

kachkaev commented Mar 8, 2022

I’ve just looked a blog post on TypeScript 4.6 — it does not seem to have any new syntax. How about mentioning this anyhow in the blog? We usually release a new minor version of Prettier after a new TypeScript release to support new syntax. If we don't say anything, some folks may think that we’ve forgotten about TS 4.6 and will go checking TS blog themselves.

@sosukesuzuki sosukesuzuki marked this pull request as ready for review March 13, 2022 08:24
changelog_unreleased/blog-post-intro.md Outdated Show resolved Hide resolved
changelog_unreleased/blog-post-intro.md Outdated Show resolved Hide resolved
};
```

And for TS that comment would not be stable with a second format:
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
And for TS that comment would not be stable with a second format:
This change also fixes an issue where a similarly-placed comment in TypeScript would be moved again after a second formatting operation:

(still not 100% happy with this wording but I think it might be clearer)

website/blog/2022-03-13-2.6.0.md Outdated Show resolved Hide resolved
website/blog/2022-03-13-2.6.0.md Outdated Show resolved Hide resolved
website/blog/2022-03-13-2.6.0.md Outdated Show resolved Hide resolved
@@ -49,6 +49,7 @@ const categories = [
{ dir: "yaml", title: "YAML" },
{ dir: "api", title: "API" },
{ dir: "cli", title: "CLI" },
{ dir: "others", title: "Others" },
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
{ dir: "others", title: "Others" },
{ dir: "other", title: "Other" },

Copy link
Member

Choose a reason for hiding this comment

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

misc

Copy link
Member

Choose a reason for hiding this comment

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

ooh misc/“Miscellaneous” is good too!

changelog_unreleased/others/12055.md Outdated Show resolved Hide resolved
changelog_unreleased/typescript/12400.md Outdated Show resolved Hide resolved
changelog_unreleased/typescript/12400.md Outdated Show resolved Hide resolved
@sosukesuzuki
Copy link
Member Author

@j-f1 Thanks for all the your reviews, every time! It is really helpful.

@@ -8,6 +8,6 @@ This release includes a new `singleAttributePerLine` option. This is an option t

We've also added support formatting for some new JavaScript syntax proposals via Babel.

If you enjoy Prettier and would like to support our work, consider sponsoring us directly via [our OpenCollective](https://opencollective.com/prettier) or by sponsoring the projects we depend on, including [typescript-eslint](https://opencollective.com/typescript-eslint), [remark](https://opencollective.com/unified), and [Babel](https://opencollective.com/babel).
If you enjoy Prettier and would like to support our work, consider sponsoring us directly via [our OpenCollective](https://opencollective.com/prettier) or by sponsoring the projects we depend on, including [typescript-eslint](https://opencollective.com/typescript-eslint), [remark](https://opencollective.com/unified), and [Babel](https://opencollective.com/babel). Also thanks to [LINE](https://line.me/en/) and [Ubie](https://ubie.life/en) for actually sponsoring us! For more information on LINE's OSS sponsoring, please read the blog post [LINE has started to donate to Prettier – why companies should support OSS](https://engineering.linecorp.com/en/blog/support-prettier-and-oss/).
Copy link
Member

@fisker fisker Mar 14, 2022

Choose a reason for hiding this comment

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

Suggested change
If you enjoy Prettier and would like to support our work, consider sponsoring us directly via [our OpenCollective](https://opencollective.com/prettier) or by sponsoring the projects we depend on, including [typescript-eslint](https://opencollective.com/typescript-eslint), [remark](https://opencollective.com/unified), and [Babel](https://opencollective.com/babel). Also thanks to [LINE](https://line.me/en/) and [Ubie](https://ubie.life/en) for actually sponsoring us! For more information on LINE's OSS sponsoring, please read the blog post [LINE has started to donate to Prettier – why companies should support OSS](https://engineering.linecorp.com/en/blog/support-prettier-and-oss/).
If you enjoy Prettier and would like to support our work, consider sponsoring us directly via [our OpenCollective](https://opencollective.com/prettier) or by sponsoring the projects we depend on, including [typescript-eslint](https://opencollective.com/typescript-eslint), [remark](https://opencollective.com/unified), and [Babel](https://opencollective.com/babel). Also thanks to some amazing sponsors such as [LINE Corporation](https://opencollective.com/line), [Airbnb](https://opencollective.com/airbnb), [Nx (by Nrwl)](https://opencollective.com/nx), [Ubie](https://opencollective.com/ubie-inc), and many others for sponsoring us! For more information on LINE's OSS sponsoring, please read the blog post [LINE has started to donate to Prettier – why companies should support OSS](https://engineering.linecorp.com/en/blog/support-prettier-and-oss/).

Let's link to their opencollective account?

Copy link
Member Author

Choose a reason for hiding this comment

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

I don't think we should link to their OpenCollective. This is because the company that sponsored us would want us to see the company's website rather than the OpenCollective link.

Copy link
Member

Choose a reason for hiding this comment

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

They have their production links on OpenCollective account.

Anyway, can you add "Airbnb" and "Nx"? They are doing generous donation monthly.

Copy link
Member Author

Choose a reason for hiding this comment

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

Anyway, can you add "Airbnb" and "Nx"? They are doing generous donation monthly.

Sure!

Btw, as you mentioned in Discord, I think we should add sponsors in our website.

@sosukesuzuki
Copy link
Member Author

@j-f1 Could you review blog-post-intro.md again?

Copy link
Member

@j-f1 j-f1 left a comment

Choose a reason for hiding this comment

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

Last few small tweaks, otherwise LGTM!

scripts/draft-blog-post.mjs Outdated Show resolved Hide resolved
website/blog/2022-03-14-2.6.0.md Outdated Show resolved Hide resolved
website/blog/2022-03-14-2.6.0.md Outdated Show resolved Hide resolved
Copy link
Member

@fisker fisker left a comment

Choose a reason for hiding this comment

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

Good job!

changelog_unreleased/blog-post-intro.md Outdated Show resolved Hide resolved
@@ -74,7 +84,7 @@ fs.writeFileSync(
replaceVersions(
[
fs.readFileSync(introFile, "utf8").trim(),
"<!--truncate-->",
shouldPrintTruncate ? "<!--truncate-->" : "",
Copy link
Member

Choose a reason for hiding this comment

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

Why not check if blog-post-intro.md contains <!--truncate-->

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks! 17ba6d8

@kachkaev kachkaev added this to the 2.6 milestone Mar 15, 2022
@kachkaev
Copy link
Member

Let’s rename the file:

-website/blog/2022-03-15-2.6.0.md
+website/blog/2022-03-16-2.6.0.md

Based on: #12410 (comment)

@sosukesuzuki sosukesuzuki merged commit eb671dd into prettier:main Mar 16, 2022
@sosukesuzuki sosukesuzuki deleted the blog-post-2.6 branch March 16, 2022 04:25
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 17, 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.

None yet

4 participants