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

feat: input-requires-label - All inputs require a label #159

Merged
merged 1 commit into from
May 16, 2020

Conversation

olore
Copy link
Contributor

@olore olore commented Jul 14, 2016

@thedaviddias thedaviddias changed the base branch from master to develop August 28, 2018 12:09
@thedaviddias thedaviddias added the triage An HTMLHint team member will look at this issue label Aug 28, 2018
@olore
Copy link
Contributor Author

olore commented Sep 3, 2018

I tried to fix the conflict, but because the version in the package.json is now 0.0.0-development the grunt command is generating lib/htmlhint.js with that version, which I'm guessing we don't want to deliver.

I think package.json version needs to change, should I change it in this PR ?

@thedaviddias
Copy link
Member

Hi @olore, I'm actually working on that. I'll publish an update in the next hour.

@olore
Copy link
Contributor Author

olore commented Sep 3, 2018

I just saw #259 and now I'm up to speed. Thanks for reviving this project!

@thedaviddias
Copy link
Member

You're welcome @olore, don't hesitate to join the discord channel to exchange about the project!

@htmlhint htmlhint deleted a comment from coveralls May 14, 2020
@thedaviddias thedaviddias changed the title New rule: input-requires-label - All inputs require a label feat: input-requires-label - All inputs require a label May 14, 2020
@thedaviddias
Copy link
Member

@olore do you want to solve the merge conflict or do you want me to fix this in a new PR?

Copy link
Member

@thedaviddias thedaviddias left a comment

Choose a reason for hiding this comment

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

Almost 4 years :D thanks for this new PR!

@thedaviddias thedaviddias added accepted and removed triage An HTMLHint team member will look at this issue labels May 16, 2020
@thedaviddias thedaviddias merged commit 9914725 into htmlhint:develop May 16, 2020
@Shinigami92
Copy link
Contributor

@olore 💪❤

github-actions bot pushed a commit that referenced this pull request May 18, 2020
# [0.13.0](v0.12.2...v0.13.0) (2020-05-18)

### Bug Fixes

* add prettier and eslint ([#388](#388)) ([aba3249](aba3249))
* add semantic release test ([#399](#399)) ([616f9cd](616f9cd))
* ignore PULL_REQUEST_TEMPLATE.md ([30b0af2](30b0af2))
* use yml in semantic.yml ([86f6700](86f6700))

### Features

* Add tags checking rule - allows specify rules for any tag and validate that ([#384](#384)) ([475aaca](475aaca))
* added attr-no-unnecessary-whitespace rule ([#385](#385)) ([03bfd4f](03bfd4f))
* new rule: input-requires-label - All inputs require a label ([#159](#159)) ([5bd40fb](5bd40fb))
* new website for htmlhint.com ([#395](#395)) ([5d0d95f](5d0d95f))
thedaviddias added a commit that referenced this pull request Nov 11, 2020
Relates to #159

Signed-off-by: Antoine Leblanc <antoine.leblanc@ovhcloud.com>

Co-authored-by: David Dias <thedaviddias@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants