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.

Sign up for GitHub

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

Jump to bottom

TRIVIAL: mod removed as useless #347

Merged
merged 1 commit into from
Sep 21, 2016
Merged

TRIVIAL: mod removed as useless #347

merged 1 commit into from
Sep 21, 2016

Conversation

miripiruni
Copy link
Contributor

@miripiruni miripiruni commented Sep 13, 2016 •
edited
Loading

BEMHTML: Entity.mod is useless.

See 6280278#diff-bad194964bfd200986dd4bcaf15c2303R20

@coveralls
Copy link

coveralls commented Sep 13, 2016 •
edited
Loading

Coverage Status

Coverage decreased (-0.002%) to 97.03% when pulling 1e4e0d7 on del-mod into 49a3e10 on master.

@tadatuta
Copy link
Member

tadatuta commented Sep 20, 2016 •
edited
Loading

is was first introduced at the very beginning of new bem-xjst (with drop of xjst as a base).
see a60a137#diff-fbdcee1674f0de2a01d9d8fe5624e063R17

@miripiruni miripiruni added this to the 7.3.0 milestone Sep 21, 2016
@miripiruni miripiruni merged commit 53dbfef into master Sep 21, 2016
@miripiruni miripiruni deleted the del-mod branch September 21, 2016 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
semver: patch
Projects
None yet
Milestone
7.3.0
Development

Successfully merging this pull request may close these issues.
3 participants
@miripiruni @coveralls @tadatuta