Skip to content

1.11 Release Notes and Changelog#4035

Merged
suchipi merged 17 commits intoprettier:masterfrom
suchipi:1_11_release
Feb 27, 2018
Merged

1.11 Release Notes and Changelog#4035
suchipi merged 17 commits intoprettier:masterfrom
suchipi:1_11_release

Conversation

@suchipi
Copy link
Copy Markdown
Member

@suchipi suchipi commented Feb 23, 2018

Here is the proposed Release Notes blog post and Changelog for the 1.11 release. Could someone please proofread it?

Rendered version here:
https://deploy-preview-4035--prettier.netlify.com/blog/2018/02/26/1.11.0.html

@suchipi suchipi mentioned this pull request Feb 23, 2018
24 tasks
@vjeux
Copy link
Copy Markdown
Contributor

vjeux commented Feb 23, 2018

Just read through it all, awesome job @suchipi writing the release notes, it's been really nice to read them through!

Also, omg @evilebottnawi you've been on fire!

@suchipi
Copy link
Copy Markdown
Member Author

suchipi commented Feb 23, 2018

@vjeux Do you have reservations about releasing on a Friday evening? Should we wait until Monday?

@suchipi
Copy link
Copy Markdown
Member Author

suchipi commented Feb 23, 2018

Ah thanks, forgot to announce that

@lipis
Copy link
Copy Markdown
Member

lipis commented Feb 23, 2018

We should mention Facebook's 100% #3931 (comment) and I thinks it's better to wait until Monday :)

@vjeux
Copy link
Copy Markdown
Contributor

vjeux commented Feb 23, 2018

What we can do is to publish a pre-release now so people can try it out and release it on Monday for real? So we don't break everyone if we have some unforeseen issue

@lipis
Copy link
Copy Markdown
Member

lipis commented Feb 23, 2018

Already mentioned.. :) Shall we wait until Monday?

@suchipi
Copy link
Copy Markdown
Member Author

suchipi commented Feb 23, 2018

I'll publish a prerelease via tag "next"

@suchipi
Copy link
Copy Markdown
Member Author

suchipi commented Feb 23, 2018

(And let's wait to merge this PR until we do the real release)

Comment thread website/blog/2018-02-23-1.11.0.md Outdated

### Facebook is 💯% Prettier

Another exciting announcement is that all the JavaScript files in the Facebook codebase have now been converted to prettier! What's mindblowing is that there hasn't been a concerted push to convert the codebase to prettier, we just added the tooling in place and documentation on how to migrate a folder to prettier and people just started converting their piece of the codebase.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

prettier -> Prettier

@suchipi
Copy link
Copy Markdown
Member Author

suchipi commented Feb 24, 2018

@lipis how do I tweet from the PrettierCode twitter account?

@suchipi
Copy link
Copy Markdown
Member Author

suchipi commented Feb 24, 2018

Or @vjeux you can redo that tweet from your account if you want, either way is fine as far as I'm concerned

@vjeux
Copy link
Copy Markdown
Contributor

vjeux commented Feb 24, 2018

https://twitter.com/Vjeux/status/967193354273308672

const abc = props.something
? xyz
: {
prop1: a,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why 3 tabs here?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

So that it's aligned when tabs are rendered as two spaces, I believe.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

There's a lot of discussion about it here: #3745

@lipis
Copy link
Copy Markdown
Member

lipis commented Feb 24, 2018

@suchipi send me your Twitter account and I'll give you access..

@suchipi
Copy link
Copy Markdown
Member Author

suchipi commented Feb 24, 2018

It's @suchipi :)

@lipis
Copy link
Copy Markdown
Member

lipis commented Feb 24, 2018

Will do it tomorrow.. not easy from mobile.. :)

Comment thread website/blog/2018-02-23-1.11.0.md Outdated

## Highlights

### CSS/SCSS/LESS
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LESS -> Less

Comment thread website/blog/2018-02-23-1.11.0.md Outdated
### CSS/SCSS/Less

#### Lists of lists now have prettier formatting ([#3930] by [@evilebottnawi])
#### Lists of lists now have Prettier formatting ([#3930] by [@evilebottnawi])
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This should still be prettier.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sure.. but why?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The PR improved the formatting of lists of lists, but we already formatted them.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

and that's why it should be written prettier instead of Prettier or am I missing something?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes that’s the reason.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I guess it’s because prettier = better here, it’s not a library name in this context.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

might even be worth using a different adjective to avoid confusion:

"Lists of lists now have cleaner formatting"
"Lists of lists now have better formatting"

or something similar

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Maybe ‘more consistent’ or just ‘consistent’? ‘Better’ is way too subjective IMO, yet ‘cleaner’ looks OK.

Agree about the confusion with `prettier’ here.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Now I realised why I was confused.. :)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Changed it to cleaner.. if anyone has a better one let me know.

@lipis lipis added this to the 1.11 milestone Feb 26, 2018
@suchipi
Copy link
Copy Markdown
Member Author

suchipi commented Feb 26, 2018

I'm busy at work today, if someone else is able to publish the release I'd appreciate it. Otherwise I'll do it this afternoon. The date on this should be changed from 2/23 to 2/26 before merge.

@ericanderson
Copy link
Copy Markdown
Contributor

Can the release wait for #4052 ?

@orta
Copy link
Copy Markdown
Contributor

orta commented Feb 26, 2018

Great work @suchipi

@lipis
Copy link
Copy Markdown
Member

lipis commented Feb 26, 2018

We need to update the post with the #4028 changes and with #4052 if it's going to make it to the release.

@lipis
Copy link
Copy Markdown
Member

lipis commented Feb 26, 2018

@ericanderson
Copy link
Copy Markdown
Contributor

#4052 doesn't need a changelog entry. It fixes a regression I caused in #4005

}
```

### JavaScript
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm not sure how granular you want to be with these release notes, but my define() tweak I thiiiink made it into this release: #3830

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Thanks, I must have missed that one. I'll add an entry for it

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you, and thanks for all your hard work!

@suchipi suchipi merged commit b1b504f into prettier:master Feb 27, 2018
@lock lock Bot added the locked-due-to-inactivity Please open a new issue and fill out the template instead of commenting. label Jan 18, 2019
@lock lock Bot locked as resolved and limited conversation to collaborators Jan 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

locked-due-to-inactivity Please open a new issue and fill out the template instead of commenting.

Projects

None yet

Development

Successfully merging this pull request may close these issues.