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(create-element): remove pfelement decorator #2305

Merged
merged 1 commit into from Jan 10, 2023

Conversation

bennypowers
Copy link
Member

What I did

  • remove @pfelement decorator from create-element templates
  • remove static version string from same
  • update demo css and js

closes RedHat-UX/red-hat-design-system#577

@bennypowers bennypowers self-assigned this Jan 10, 2023
@changeset-bot
Copy link

changeset-bot bot commented Jan 10, 2023

🦋 Changeset detected

Latest commit: 0fbfc5d

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

This PR includes changesets to release 1 package
Name Type
@patternfly/create-element Patch

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

@bennypowers bennypowers enabled auto-merge (rebase) January 10, 2023 10:53
@github-actions github-actions bot added generator Updates relating to the generator tools Development and build tools labels Jan 10, 2023
@github-actions github-actions bot added this to In progress in Workflow Jan 10, 2023
@github-actions
Copy link
Contributor

Deploy Preview for patternfly-elements ready!

Name Link
🔨 Latest commit 81be645
😎 Deploy Preview https://deploy-preview-2305--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 Jan 10, 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.

I think this is an appropriate fix given we are moving toward single-project version publishing. When I had explored this issue earlier I went down the rabbit hole of making the generation templates customizable in a child repo. We can maybe explore that as well in a future PR.

Lackluster Greek Takeout, Moussaka

Workflow automation moved this from In progress to Approved Jan 10, 2023
@bennypowers bennypowers merged commit fdf12c7 into main Jan 10, 2023
19 checks passed
Workflow automation moved this from Approved to Done Jan 10, 2023
@bennypowers bennypowers deleted the fix/create/remove-pfelement branch January 10, 2023 15:37
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 generator Updates relating to the generator ready to merge tools Development and build tools
Projects
Development

Successfully merging this pull request may close these issues.

[feat] Remove @pfelement decorator from element init
2 participants