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(purgecss) #905

Merged
merged 7 commits into from
Jan 2, 2022
Merged

fix(purgecss) #905

merged 7 commits into from
Jan 2, 2022

Conversation

kellymears
Copy link
Member

Type of change

  • PATCH: bugfix

Dependencies added

  • none

Details

  • fix: issue with bud.purgecss fn bindings
  • add: unit tests
  • add: integration tests
  • add: examples/purgecss

🥳 this dude has 100% coverage now

@kellymears kellymears added this to the 5.1.0 milestone Jan 1, 2022
@kellymears kellymears self-assigned this Jan 1, 2022
@github-actions github-actions bot added @roots/bud-purgecss dependencies Pull requests that update a dependency file examples test labels Jan 1, 2022
/**
* Module registration
*
* @todo facade bindings to `@roots/bud-extension`
Copy link
Member Author

Choose a reason for hiding this comment

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

The only reason for this register callback is that I need to set/bind the facade for bud.config.js.

this should eventually be moved to @roots/bud-extensions/src/Controller/controller.service.ts

Copy link
Member Author

Choose a reason for hiding this comment

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

addressing this in #906. let's get that merged first and then I can remove this.

@kellymears kellymears added this to Review in Project releases Jan 1, 2022
@kellymears kellymears merged commit 008adbb into next Jan 2, 2022
@kellymears kellymears deleted the fix-purgecss branch January 2, 2022 06:40
@kellymears kellymears mentioned this pull request Jan 4, 2022
3 tasks
@kellymears kellymears moved this from Review to next in Project releases Jan 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file examples @roots/bud-purgecss test
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

1 participant