Skip to content

Upgrading @primer/stylelint-config@12.3.3 - #1956

Merged
jonrohan merged 5 commits into
mainfrom
upgrade-stylelint-config
Feb 28, 2022
Merged

Upgrading @primer/stylelint-config@12.3.3#1956
jonrohan merged 5 commits into
mainfrom
upgrade-stylelint-config

Conversation

@jonrohan

@jonrohanjonrohan commented Feb 27, 2022

Copy link
Copy Markdown
Member

What are you trying to accomplish?

This upgrades @primer/stylelint-config to 12.3.2 and 12.3.3 which have some improvements to the spacing plugin to check functional groups.

What approach did you choose and why?

yarn add @primer/stylelint-config@latest
yarn stylelint:fix
yarn stylelint:add-disables

Are additional changes needed?

  • No, this PR should be ok to ship as is. 🚢

@changeset-bot

changeset-botBot commented Feb 27, 2022

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 2bdae2f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
NameType
@primer/cssPatch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@jonrohanjonrohan changed the title Upgrading @primer/stylelint-config@12.3.2Upgrading @primer/stylelint-config@12.3.3Feb 28, 2022
display: block;
height: 1px;
padding: 0;
// stylelint-disable-next-line primer/spacing

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

😃

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@jonrohan@langermank