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

Update changeset config to reflect changes from main #10253

Merged
merged 1 commit into from
Aug 28, 2023

Conversation

gwyneplaine
Copy link
Contributor

@gwyneplaine gwyneplaine commented Aug 28, 2023

WHY are these changes introduced?

#10238 merged in changes from main, inclusive of removing the polaris-cli and polaris-codemods packages. Unfortunately these changes were not reflected in the changeset config resulting in errors when running yarn changeset due to ignores being set up for the aforementioned removed packages.

Error:
Screenshot 2023-08-28 at 11 06 56 am

WHAT is this pull request doing?

  • Removes removed packages (polaris-cli polaris-codemods) from changest config ignore.

How to 🎩

Run yarn changeset locally on branch.

@gwyneplaine gwyneplaine changed the title Fix changeset config to align with main Update changeset config to reflect changes from main Aug 28, 2023
@gwyneplaine gwyneplaine merged commit 7aef08b into next Aug 28, 2023
9 checks passed
@gwyneplaine gwyneplaine deleted the update/changeset-config branch August 28, 2023 03:43
sophschneider pushed a commit that referenced this pull request Sep 19, 2023
<!--
  ☝️How to write a good PR title:
- Prefix it with [ComponentName] (if applicable), for example: [Button]
  - Start with a verb, for example: Add, Delete, Improve, Fix…
  - Give as much context as necessary and as little as possible
  - Prefix it with [WIP] while it’s a work in progress
-->

### WHY are these changes introduced?
#10238 merged in changes from main, inclusive of removing the
`polaris-cli` and `polaris-codemods` packages. Unfortunately these
changes were not reflected in the changeset config resulting in errors
when running `yarn changeset` due to ignores being set up for the
aforementioned removed packages.

Error:
![Screenshot 2023-08-28 at 11 06 56
am](https://github.com/Shopify/polaris/assets/12119389/143d9e19-375d-4f37-a591-dd2d46511cca)


<!--
  Context about the problem that’s being addressed.
-->

### WHAT is this pull request doing?

* Removes removed packages (`polaris-cli` `polaris-codemods`) from
changest config ignore.

<!-- ℹ️ Delete the following for small / trivial changes -->

### How to 🎩

Run `yarn changeset` locally on branch.
AnnaCheba pushed a commit to AnnaCheba/polaris that referenced this pull request Apr 22, 2024
<!--
  ☝️How to write a good PR title:
- Prefix it with [ComponentName] (if applicable), for example: [Button]
  - Start with a verb, for example: Add, Delete, Improve, Fix…
  - Give as much context as necessary and as little as possible
  - Prefix it with [WIP] while it’s a work in progress
-->

### WHY are these changes introduced?
Shopify#10238 merged in changes from main, inclusive of removing the
`polaris-cli` and `polaris-codemods` packages. Unfortunately these
changes were not reflected in the changeset config resulting in errors
when running `yarn changeset` due to ignores being set up for the
aforementioned removed packages.

Error:
![Screenshot 2023-08-28 at 11 06 56
am](https://github.com/Shopify/polaris/assets/12119389/143d9e19-375d-4f37-a591-dd2d46511cca)


<!--
  Context about the problem that’s being addressed.
-->

### WHAT is this pull request doing?

* Removes removed packages (`polaris-cli` `polaris-codemods`) from
changest config ignore.

<!-- ℹ️ Delete the following for small / trivial changes -->

### How to 🎩

Run `yarn changeset` locally on branch.
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.

None yet

1 participant