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

fix(icon)!: remove BaseIcon #2636

Open
wants to merge 6 commits into
base: staging/4.0
Choose a base branch
from
Open

Conversation

bennypowers
Copy link
Member

Closes #2621

What I did

  1. remove baseicon

Testing Instructions

  1. dp demo vs prod

Notes to Reviewers

  1. don't merge until non-breaking prs are merged and 2.5.0 is released
  2. take special care to review JSDoc: slot, csspart, cssprop

@changeset-bot
Copy link

changeset-bot bot commented Oct 19, 2023

🦋 Changeset detected

Latest commit: d462f0d

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

This PR includes changesets to release 1 package
Name Type
@patternfly/elements Major

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

@github-actions github-actions bot added functionality Functionality, typically pertaining to the JavaScript. styles An issue or PR pertaining only to CSS/Sass labels Oct 19, 2023
@github-actions github-actions bot added this to In progress in Workflow Oct 19, 2023
@netlify
Copy link

netlify bot commented Oct 19, 2023

Deploy Preview for patternfly-elements ready!

Name Link
🔨 Latest commit 758dac4
😎 Deploy Preview https://deploy-preview-2636--patternfly-elements.netlify.app/

To edit notification comments on pull requests, go to your Netlify site settings.

@github-actions github-actions bot added the AT passed Automated testing has passed label Oct 19, 2023
Copy link
Collaborator

@zeroedin zeroedin left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -0,0 +1,4 @@
---
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
---
---

Ignore suggestion, blocking comment for approval.

Don't merge until non-breaking changes are released in 2.5.0

Workflow automation moved this from In progress to Approved Nov 21, 2023
bennypowers and others added 2 commits November 22, 2023 09:58
Co-authored-by: Steven Spriggs <steven.spriggs@gmail.com>
Co-authored-by: Steven Spriggs <steven.spriggs@gmail.com>
@bennypowers bennypowers added the pfe 4.0 For PFE version 4.0 (breaking change) label Mar 22, 2024
@bennypowers bennypowers changed the base branch from main to staging/4.0 March 31, 2024 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AT passed Automated testing has passed functionality Functionality, typically pertaining to the JavaScript. pfe 4.0 For PFE version 4.0 (breaking change) ready to merge styles An issue or PR pertaining only to CSS/Sass
Projects
Workflow
Approved
Development

Successfully merging this pull request may close these issues.

Remove BaseIcon
2 participants