Skip to content

Break if string | null#2028

Merged
vjeux merged 1 commit intoprettier:masterfrom
vjeux:break_string_or_null
Jun 7, 2017
Merged

Break if string | null#2028
vjeux merged 1 commit intoprettier:masterfrom
vjeux:break_string_or_null

Conversation

@vjeux
Copy link
Copy Markdown
Contributor

@vjeux vjeux commented Jun 7, 2017

We were a bit too aggressive around the | null heuristic.

Fixes #2002

We were a bit too aggressive around the `| null` heuristic.

Fixes prettier#2002
@vjeux vjeux merged commit c7cacf0 into prettier:master Jun 7, 2017
blowery added a commit to Automattic/wp-prettier that referenced this pull request Jun 21, 2017
* tag '1.4.4': (421 commits)
  1.4.4
  1.4.3
  Clean up build script, add Babel for docs js bundles (prettier#1970)
  Check if the next group has comments (prettier#2040)
  Include typeAnnotation inside of node location (prettier#2039)
  Fix decorator comments for export class (prettier#2038)
  Add group around TSMethodSignature (prettier#2037)
  Do not throw for unprinted comments inside of JSX (prettier#2036)
  Break decorator before content (prettier#2032)
  Fix less &:extends when parsed with scss (prettier#2034)
  Fix parenthesis for call expression inside of member expression inside of new (prettier#2035)
  Break if string | null (prettier#2028)
  Only prevent lone `set` if there's no type annotations (prettier#2026)
  Add semi for functions without body (prettier#2025)
  Update commands.md
  Update commands.md
  Pretty printer commands.js
  GraphQL: Add support for union types (prettier#2014)
  Update README for emacs integration link (prettier#2024)
  Add alias and null value (prettier#2017)
  ...
@lock lock Bot added the locked-due-to-inactivity Please open a new issue and fill out the template instead of commenting. label Jan 20, 2019
@lock lock Bot locked as resolved and limited conversation to collaborators Jan 20, 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.

1 participant