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

added barrel file generation #1015

Merged
merged 1 commit into from
Nov 21, 2024
Merged

added barrel file generation #1015

merged 1 commit into from
Nov 21, 2024

Conversation

shairez
Copy link
Contributor

@shairez shairez commented Nov 21, 2024

What is it?

  • Feature / enhancement
  • Bug
  • Docs / tests
  • Other

Why is it needed?

This makes the styled-kit much easier to import components from.

Every generated file is added to the barrel file (index.ts) on the root of the component folder.

That way you can just import all generated styled components from a single point instead of ;/input/input etc.

Checklist:

  • My code follows the developer guidelines of this project
  • I have performed a self-review of my own code
  • I have ran pnpm change and documented my changes
  • Added new tests to cover the fix / functionality

Copy link

changeset-bot bot commented Nov 21, 2024

🦋 Changeset detected

Latest commit: 336ac8d

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

This PR includes changesets to release 1 package
Name Type
qwik-ui Minor

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

Copy link

pkg-pr-new bot commented Nov 21, 2024

Open in Stackblitz

pnpm add https://pkg.pr.new/qwikifiers/qwik-ui@1015
pnpm add https://pkg.pr.new/qwikifiers/qwik-ui/@qwik-ui/headless@1015
pnpm add https://pkg.pr.new/qwikifiers/qwik-ui/@qwik-ui/utils@1015
pnpm add https://pkg.pr.new/qwikifiers/qwik-ui/@qwik-ui/styled@1015

commit: 336ac8d

@shairez
Copy link
Contributor Author

shairez commented Nov 21, 2024

tested this PR on one of my projects... it works 😊

merging

@shairez shairez merged commit 57d9286 into main Nov 21, 2024
6 checks passed
@shairez shairez deleted the pr-add-barrel branch November 21, 2024 00:25
@github-actions github-actions bot mentioned this pull request Nov 20, 2024
@shairez shairez self-assigned this Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

1 participant