Skip to content

Conversation

@pldg
Copy link
Contributor

@pldg pldg commented Jul 20, 2019

  • Using cssAttributes with rel property defined doesn't output rel="stylesheet" by default anymore
  • You can now set css or js attributes to an empty string to output only the attribute name (without ="")
  • Update devDependencies

- Using `cssAttributes` with `rel` property defined doesn't output `rel="stylesheet"` by default anymore
- You can now set css or js attributes to an empty string to output only the attribute name (without `=""`)
- Update devDependencies
Copy link
Member

@sapegin sapegin left a comment

Choose a reason for hiding this comment

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

Thanks for the pull request!

A have a few comments, and please revert all changes to package files. Tests would be great too!

Some tests need revisions:

- Test for `cssAttributes` doesn't add link tag in html
- Test for `filename` should output a file
@bebraw
Copy link
Collaborator

bebraw commented Dec 6, 2019

@sapegin Let's merge and publish?

Copy link
Member

@sapegin sapegin left a comment

Choose a reason for hiding this comment

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

Please ping me when it's ready for another look, I'd never know about that otherwise.

@@ -0,0 +1 @@
import './style.css';
Copy link
Member

Choose a reason for hiding this comment

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

Let's move these files to some folder so it's clear they are test files, they shouldn't be in the root folder.

@sapegin
Copy link
Member

sapegin commented Dec 6, 2019

@bebraw As soon as it's ready.

bebraw added a commit that referenced this pull request Jan 29, 2020
@bebraw bebraw closed this in #22 Jan 29, 2020
bebraw added a commit that referenced this pull request Jan 29, 2020
@sapegin
Copy link
Member

sapegin commented Jan 29, 2020

🎉 This issue has been resolved in version 2.2.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

3 participants