Skip to content

Conversation

@sam-b-rose
Copy link
Member

The current migrations would inject a migration comment when encountering any numeric operator in a style declaration.

This change adds an early exit if the target Sass function ins't present in the declaration.

@sam-b-rose
Copy link
Member Author

/snapit

@github-actions
Copy link
Contributor

github-actions bot commented Jan 4, 2023

🫰✨ Thanks @samrose3! Your snapshots have been published to npm.

Test the snapshots by updating your package.json with the newly published versions:

yarn add @shopify/plugin-polaris@0.0.0-snapshot-release-20230104003525
yarn add @shopify/polaris-migrator@0.0.0-snapshot-release-20230104003525
yarn add @shopify/polaris@0.0.0-snapshot-release-20230104003525
yarn add @shopify/stylelint-polaris@0.0.0-snapshot-release-20230104003525

@sam-b-rose sam-b-rose merged commit bfb5377 into main Jan 4, 2023
@sam-b-rose sam-b-rose deleted the fix-duration-migration branch January 4, 2023 00:56
aveline pushed a commit that referenced this pull request Jan 4, 2023
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @shopify/polaris@10.17.0

### Minor Changes

- [#7408](#7408)
[`7ffd87f7d`](7ffd87f)
Thanks [@laurkim](https://github.com/laurkim)! - - Refactored
`ChoiceList` to use primitive Layout components
    -   Added support for `legend` element to `Box`
    -   Added support for `fieldset` element to `AlphaStack`


- [#7978](#7978)
[`fb0ed3805`](fb0ed38)
Thanks [@kyledurand](https://github.com/kyledurand)! - Added
`printHidden` prop to `Box`


- [#7408](#7408)
[`7ffd87f7d`](7ffd87f)
Thanks [@laurkim](https://github.com/laurkim)! - Updated `Banner`
component to use new layout primitives


- [#7408](#7408)
[`7ffd87f7d`](7ffd87f)
Thanks [@laurkim](https://github.com/laurkim)! - Refactored `Modal` and
its children components to use layout primitives


- [#7963](#7963)
[`f94cf1496`](f94cf14)
Thanks [@aveline](https://github.com/aveline)! - Updated `AlphaStack`
docs for `align` prop


- [#7408](#7408)
[`7ffd87f7d`](7ffd87f)
Thanks [@laurkim](https://github.com/laurkim)! - Refactored
`AccountConnection` to use new layout primitives


- [#7915](#7915)
[`81fd3fd5b`](81fd3fd)
Thanks [@melaniedamilig](https://github.com/melaniedamilig)! - - Added
the `onAnimationEnd` prop to `Collapsible`
- Fixed a bug in `Filters` where focus was moved to collapsed filter
contents before the `Collapsible` animation ended


- [#7956](#7956)
[`30cdd2e23`](30cdd2e)
Thanks [@aveline](https://github.com/aveline)! - Updated `Box` allowable
aria roles


- [#7939](#7939)
[`8b31e3983`](8b31e39)
Thanks [@acidio](https://github.com/acidio)! - Added support for
`spacing` prop to List component allowing for a more compact list

### Patch Changes

- [#7925](#7925)
[`4e33e1ced`](4e33e1c)
Thanks [@jas7457](https://github.com/jas7457)! - Updated `IndexTable`
and `ProgressBar` to no longer log errors about deprecated
`React.findDOMNode`

## @shopify/plugin-polaris@0.0.23

### Patch Changes

- Updated dependencies
\[[`bfb537780`](bfb5377)]:
    -   @shopify/polaris-migrator@0.10.2

## @shopify/polaris-migrator@0.10.2

### Patch Changes

- [#7979](#7979)
[`bfb537780`](bfb5377)
Thanks [@samrose3](https://github.com/samrose3)! - Early exit if target
function isn't present for the scss-replace-duration and
scss-replace-easing migrations

- Updated dependencies
\[[`af0ceb8c6`](af0ceb8),
[`e7712e7a5`](e7712e7)]:
    -   @shopify/stylelint-polaris@5.0.2

## @shopify/stylelint-polaris@5.0.2

### Patch Changes

- [#7954](#7954)
[`af0ceb8c6`](af0ceb8)
Thanks [@aaronccasanova](https://github.com/aaronccasanova)! - Use
RegExp pattern to exclude reporting invalid scope disables and address
memory issues


- [#7919](#7919)
[`e7712e7a5`](e7712e7)
Thanks [@qt314](https://github.com/qt314)! - Deduped converage rules
that were in multiple categories

## polaris.shopify.com@0.28.0

### Minor Changes

- [#7963](#7963)
[`f94cf1496`](f94cf14)
Thanks [@aveline](https://github.com/aveline)! - Updated `AlphaStack`
docs for `align` prop

### Patch Changes

- [#7921](#7921)
[`502530597`](5025305)
Thanks [@martenbjork](https://github.com/martenbjork)! - Fixed border
radius of component images in the overview page.

- Updated dependencies
\[[`7ffd87f7d`](7ffd87f),
[`fb0ed3805`](fb0ed38),
[`7ffd87f7d`](7ffd87f),
[`7ffd87f7d`](7ffd87f),
[`f94cf1496`](f94cf14),
[`4e33e1ced`](4e33e1c),
[`7ffd87f7d`](7ffd87f),
[`81fd3fd5b`](81fd3fd),
[`30cdd2e23`](30cdd2e),
[`8b31e3983`](8b31e39)]:
    -   @shopify/polaris@10.17.0

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@gwyneplaine gwyneplaine mentioned this pull request Feb 14, 2023
juzser pushed a commit to juzser/polaris that referenced this pull request Jul 27, 2023
…and `scss-replace-easing`) (Shopify#7979)

The current migrations would inject a migration comment when
encountering any numeric operator in a style declaration.

This change adds an early exit if the target Sass function ins't present
in the declaration.
juzser pushed a commit to juzser/polaris that referenced this pull request Jul 27, 2023
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @shopify/polaris@10.17.0

### Minor Changes

- [Shopify#7408](Shopify#7408)
[`7ffd87f7d`](Shopify@7ffd87f)
Thanks [@laurkim](https://github.com/laurkim)! - - Refactored
`ChoiceList` to use primitive Layout components
    -   Added support for `legend` element to `Box`
    -   Added support for `fieldset` element to `AlphaStack`


- [Shopify#7978](Shopify#7978)
[`fb0ed3805`](Shopify@fb0ed38)
Thanks [@kyledurand](https://github.com/kyledurand)! - Added
`printHidden` prop to `Box`


- [Shopify#7408](Shopify#7408)
[`7ffd87f7d`](Shopify@7ffd87f)
Thanks [@laurkim](https://github.com/laurkim)! - Updated `Banner`
component to use new layout primitives


- [Shopify#7408](Shopify#7408)
[`7ffd87f7d`](Shopify@7ffd87f)
Thanks [@laurkim](https://github.com/laurkim)! - Refactored `Modal` and
its children components to use layout primitives


- [Shopify#7963](Shopify#7963)
[`f94cf1496`](Shopify@f94cf14)
Thanks [@aveline](https://github.com/aveline)! - Updated `AlphaStack`
docs for `align` prop


- [Shopify#7408](Shopify#7408)
[`7ffd87f7d`](Shopify@7ffd87f)
Thanks [@laurkim](https://github.com/laurkim)! - Refactored
`AccountConnection` to use new layout primitives


- [Shopify#7915](Shopify#7915)
[`81fd3fd5b`](Shopify@81fd3fd)
Thanks [@melaniedamilig](https://github.com/melaniedamilig)! - - Added
the `onAnimationEnd` prop to `Collapsible`
- Fixed a bug in `Filters` where focus was moved to collapsed filter
contents before the `Collapsible` animation ended


- [Shopify#7956](Shopify#7956)
[`30cdd2e23`](Shopify@30cdd2e)
Thanks [@aveline](https://github.com/aveline)! - Updated `Box` allowable
aria roles


- [Shopify#7939](Shopify#7939)
[`8b31e3983`](Shopify@8b31e39)
Thanks [@acidio](https://github.com/acidio)! - Added support for
`spacing` prop to List component allowing for a more compact list

### Patch Changes

- [Shopify#7925](Shopify#7925)
[`4e33e1ced`](Shopify@4e33e1c)
Thanks [@jas7457](https://github.com/jas7457)! - Updated `IndexTable`
and `ProgressBar` to no longer log errors about deprecated
`React.findDOMNode`

## @shopify/plugin-polaris@0.0.23

### Patch Changes

- Updated dependencies
\[[`bfb537780`](Shopify@bfb5377)]:
    -   @shopify/polaris-migrator@0.10.2

## @shopify/polaris-migrator@0.10.2

### Patch Changes

- [Shopify#7979](Shopify#7979)
[`bfb537780`](Shopify@bfb5377)
Thanks [@samrose3](https://github.com/samrose3)! - Early exit if target
function isn't present for the scss-replace-duration and
scss-replace-easing migrations

- Updated dependencies
\[[`af0ceb8c6`](Shopify@af0ceb8),
[`e7712e7a5`](Shopify@e7712e7)]:
    -   @shopify/stylelint-polaris@5.0.2

## @shopify/stylelint-polaris@5.0.2

### Patch Changes

- [Shopify#7954](Shopify#7954)
[`af0ceb8c6`](Shopify@af0ceb8)
Thanks [@aaronccasanova](https://github.com/aaronccasanova)! - Use
RegExp pattern to exclude reporting invalid scope disables and address
memory issues


- [Shopify#7919](Shopify#7919)
[`e7712e7a5`](Shopify@e7712e7)
Thanks [@qt314](https://github.com/qt314)! - Deduped converage rules
that were in multiple categories

## polaris.shopify.com@0.28.0

### Minor Changes

- [Shopify#7963](Shopify#7963)
[`f94cf1496`](Shopify@f94cf14)
Thanks [@aveline](https://github.com/aveline)! - Updated `AlphaStack`
docs for `align` prop

### Patch Changes

- [Shopify#7921](Shopify#7921)
[`502530597`](Shopify@5025305)
Thanks [@martenbjork](https://github.com/martenbjork)! - Fixed border
radius of component images in the overview page.

- Updated dependencies
\[[`7ffd87f7d`](Shopify@7ffd87f),
[`fb0ed3805`](Shopify@fb0ed38),
[`7ffd87f7d`](Shopify@7ffd87f),
[`7ffd87f7d`](Shopify@7ffd87f),
[`f94cf1496`](Shopify@f94cf14),
[`4e33e1ced`](Shopify@4e33e1c),
[`7ffd87f7d`](Shopify@7ffd87f),
[`81fd3fd5b`](Shopify@81fd3fd),
[`30cdd2e23`](Shopify@30cdd2e),
[`8b31e3983`](Shopify@8b31e39)]:
    -   @shopify/polaris@10.17.0

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant