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

Prettier 1.20 - Last 1.x release? #7675

Closed
duailibe opened this issue Feb 26, 2020 · 14 comments
Closed

Prettier 1.20 - Last 1.x release? #7675

duailibe opened this issue Feb 26, 2020 · 14 comments
Assignees
Labels
locked-due-to-inactivity Please open a new issue and fill out the template instead of commenting.

Comments

@duailibe
Copy link
Member

duailibe commented Feb 26, 2020

The latest release (1.19) was in November of last year. We have a few bugfixes that we could be released already before the 2.0 release.

I'll work on this today

@duailibe duailibe self-assigned this Feb 26, 2020
@alexander-akait
Copy link
Member

@duailibe Which PRs we want to port for 1.20?

@duailibe
Copy link
Member Author

duailibe commented Feb 26, 2020

@evilebottnawi I was gonna cut a release from master, but if there are PRs that can be ported easy enough, I'm happy to do it

@duailibe duailibe pinned this issue Feb 26, 2020
@thorn0
Copy link
Member

thorn0 commented Feb 26, 2020

I'd say #6949 and #7026. Users complain the most about these two. They are in master, BTW.

@duailibe
Copy link
Member Author

Yes, #6949 is the main reason, which is why a release from master would suffice

@alexander-akait
Copy link
Member

alexander-akait commented Feb 26, 2020

@duailibe master contains breaking changes, we can't release this as 1.20, we do not have a lot of work for 2, so I would probably rather not do release 1.20, but if all votes for 1.20, I am ok

@duailibe
Copy link
Member Author

@evilebottnawi what breaking changes?

@graemeworthy
Copy link
Contributor

graemeworthy commented Feb 26, 2020

#7654 does not rely on other items in the 2.0 branch, and can be easily ported, it is not in master.

I have rebased on master:
master...graemeworthy:prettier-ignore-should-also-ignore-text-nodes

@thorn0
Copy link
Member

thorn0 commented Feb 26, 2020

Let's merge #7660 (as is, without addressing this issue) and release 2.0 instead.

@fisker
Copy link
Member

fisker commented Feb 27, 2020

#6981

@duailibe
Copy link
Member Author

@thorn0 IMO you shouldn't rush a 2.0 release, but it's your call. I wanted to get those fixes out by today or tomorrow. Do you think 2.0 will be ready by then?

@krilllind
Copy link

Would it be possible to release a 1.19.2 patch including issues related to 1.19(.1) and then just wait with 2.0?

I'm for example waiting for this fix to be released.

@heylookltsme
Copy link
Contributor

I'd love to have this included in a minor release, if possible! 🙏

@mvlabat
Copy link

mvlabat commented Mar 16, 2020

I would also love to see the fix of this: #6953

@eps1lon
Copy link

eps1lon commented Mar 20, 2020

#7631 would be nice as well. Generally any change that deals with new syntax would be nice to backport. Otherwise you might get stuck in this weird limbo where you can't upgrade syntax because you can't ugrade prettier.

@thorn0 thorn0 closed this as completed Mar 21, 2020
@thorn0 thorn0 unpinned this issue Mar 21, 2020
@lock lock bot added the locked-due-to-inactivity Please open a new issue and fill out the template instead of commenting. label Jun 24, 2020
@lock lock bot locked as resolved and limited conversation to collaborators Jun 24, 2020
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

No branches or pull requests

9 participants