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

[sass-loader] Prepend options.data to code payload #233

Merged
merged 2 commits into from
Mar 5, 2020

Conversation

drewlustro
Copy link

Originally by @bastienrobert on #179 (currently has conflicts). I'm hoping this new PR will be friendlier to merge.

Adds data prepend feature common on sass-loader implementations.

Example: https://github.com/differui/rollup-plugin-sass#options-1

cc @cortopy @egoist

Adds `data` prepend feature common on
sass-loader implementations:
https://github.com/differui/rollup-plugin-sass#options-1

Care of @bastienrobert
@codecov
Copy link

codecov bot commented Feb 20, 2020

Codecov Report

Merging #233 into master will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #233      +/-   ##
==========================================
+ Coverage   91.69%   91.72%   +0.02%     
==========================================
  Files           9        9              
  Lines         289      290       +1     
  Branches      100      101       +1     
==========================================
+ Hits          265      266       +1     
  Misses         24       24              

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d83e425...63ee350. Read the comment docs.

Copy link
Collaborator

@himself65 himself65 left a comment

Choose a reason for hiding this comment

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

if tests added, it will be ok

@bastienrobert
Copy link

I’m going to add tests asap to merge this

@drewlustro
Copy link
Author

I’m going to add tests asap to merge this

I can also write some tests today if @bastienrobert doesn't beat me to it 🤙

@bastienrobert
Copy link

@drewlustro oh dude, It's actually 7PM here, I'm drinking a beer so be my guest ! 😁

@drewlustro
Copy link
Author

✅ Tests up! cc @bastienrobert

@bastienrobert
Copy link

Seems good to me @drewlustro! Well done and thanks for the mention :)

@himself65 himself65 merged commit 80bb9da into egoist:master Mar 5, 2020
@egoist
Copy link
Owner

egoist commented Mar 5, 2020

🎉 This PR is included in version 2.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@egoist egoist added the released label Mar 5, 2020
@drewlustro drewlustro deleted the options.data-prepend-to-payload branch March 5, 2020 00:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants