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

Adding fill because it is a valid css attribute. #150

Merged
merged 1 commit into from
Feb 2, 2016
Merged

Adding fill because it is a valid css attribute. #150

merged 1 commit into from
Feb 2, 2016

Conversation

lukasoppermann
Copy link
Contributor

No description provided.

@lukasoppermann
Copy link
Contributor Author

Is this going to be merged and released soon?

@frvge frvge added the Rebase label Jan 10, 2016
@XhmikosR XhmikosR removed the Rebase label Jan 18, 2016
cscott added a commit to cscott/parser-lib that referenced this pull request Feb 1, 2016
cscott added a commit to cscott/parser-lib that referenced this pull request Feb 2, 2016
@cscott
Copy link
Contributor

cscott commented Feb 2, 2016

@XhmikosR this patch can be directly merged. I've tested it, seems fine.

@XhmikosR XhmikosR added this to the v0.2.6 milestone Feb 2, 2016
XhmikosR added a commit that referenced this pull request Feb 2, 2016
Adding fill because it is a valid css attribute.
@XhmikosR XhmikosR merged commit ff165a2 into CSSLint:master Feb 2, 2016
@XhmikosR
Copy link
Member

XhmikosR commented Feb 2, 2016

@stubbornella @nschonni: might be worth giving @cscott push rights.

@stubbornella
Copy link
Member

Done

@XhmikosR
Copy link
Member

XhmikosR commented Feb 3, 2016

Hmm, I don't see him listed but maybe it's because he didn't accept the email yet. ping @cscott

@cscott
Copy link
Contributor

cscott commented Feb 3, 2016

Thanks, @XhmikosR and @stubbornella. You're right, it was buried in my inbox. But I found it!

I'm going to continue to push PRs for now, to ensure the patches get reviewed by someone other than myself before being merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants