Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@agilgur5
Copy link
Collaborator

@agilgur5 agilgur5 commented Aug 25, 2020

Description

  • optional-chaining and nullish-coalescing are now part of
    @babel/preset-env, so no need to explicitly install or configure
    them anymore

  • update autoprefixer integration test because browserslist and
    caniuse-lite were updated to more recent compat tables

    • -webkit-input-placeholder is no longer needed, but some others
      are, so switch to checking for -moz-placeholder
    • this probably fixes that "caniuse-lite is out-of-date" warning I
      was getting occassionally in CI
      • didn't get locally probably because I didn't install with
        --frozen-lockfile locally, unlike in CI

Tags

#370 added this shortly after the syntax landed in TS.

Misc Notes

Been meaning to do this for a while and finally got to it

@vercel

This comment has been minimized.

@agilgur5
Copy link
Collaborator Author

Huh... rollup-plugin-postcss integration test is failing now... instead of the syntax regression tests... well that's odd

@agilgur5
Copy link
Collaborator Author

Ohhh it changed the output of autoprefixer a bit... I guess because the browserslist / caniuse compat table was updated to something more reflective of today and so -webkit-input-placeholder is no longer necessary.

- optional-chaining and nullish-coalescing are now part of
  @babel/preset-env, so no need to explicitly install or configure
  them anymore
  - left in the regression tests though!
  - previously they were new syntax and TSDX adopted them as soon as
    they landed in TS
  - can view current compat table here:
    https://github.com/babel/babel/blob/e498bee10f0123bb208baa228ce6417542a2c3c4/packages/babel-compat-data/data/plugins.json#L24

- update autoprefixer integration test because browserslist and
  caniuse-lite were updated to more recent compat tables
  - `-webkit-input-placeholder` is no longer needed, but some others
    are, so switch to checking for `-moz-placeholder`
  - this probably fixes that "caniuse-lite is out-of-date" warning I
    was getting occassionally in CI
    - didn't get locally probably because I didn't install with
      `--frozen-lockfile` locally, unlike in CI
Copy link
Collaborator Author

@agilgur5 agilgur5 left a comment

Choose a reason for hiding this comment

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

Fixed autoprefixer test, all tests pass now!

@agilgur5 agilgur5 merged commit 8e2d750 into jaredpalmer:master Aug 29, 2020
paul-vd pushed a commit to EezyQuote/tsdx that referenced this pull request Dec 1, 2020
…lmer#838)

- optional-chaining and nullish-coalescing are now part of
  @babel/preset-env, so no need to explicitly install or configure
  them anymore
  - left in the regression tests though!
  - previously they were new syntax and TSDX adopted them as soon as
    they landed in TS
  - can view current compat table here:
    https://github.com/babel/babel/blob/e498bee10f0123bb208baa228ce6417542a2c3c4/packages/babel-compat-data/data/plugins.json#L24

- update autoprefixer integration test because browserslist and
  caniuse-lite were updated to more recent compat tables
  - `-webkit-input-placeholder` is no longer needed, but some others
    are, so switch to checking for `-moz-placeholder`
  - this probably fixes that "caniuse-lite is out-of-date" warning I
    was getting occassionally in CI
    - didn't get locally probably because I didn't install with
      `--frozen-lockfile` locally, unlike in CI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

scope: dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant