Skip to content

feat(client): delete none and warning from clientLogLevel - #1998

Merged
hiroppy merged 1 commit into
nextfrom
feature/modify-loglevel
Jun 10, 2019
Merged

feat(client): delete none and warning from clientLogLevel#1998
hiroppy merged 1 commit into
nextfrom
feature/modify-loglevel

Conversation

@hiroppy

Copy link
Copy Markdown
Contributor

ISSUE: #1901

  • This is a bugfix
  • This is a feature
  • This is a code refactor
  • This is a test update
  • This is a docs update
  • This is a metadata update

For Bugs and Features; did you add new tests?

yes

Motivation / Use-Case

See the title.

Breaking Changes

yes

Additional Info

@hiroppy

This comment has been minimized.

@codecov

codecovBot commented Jun 8, 2019

Copy link
Copy Markdown

Codecov Report

Merging #1998 into next will decrease coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@ Coverage Diff @@## next #1998 +/- ##
=========================================
- Coverage 92.4% 92.36% -0.05% 
=========================================
Files 28 28 Lines 1067 1061 -6 Branches 324 322 -2 =========================================
- Hits 986 980 -6 
Misses 77 77 Partials 4 4
Impacted FilesCoverage Δ
client-src/default/utils/log.js100% <ø> (ø)⬆️

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 4558b1e...c226f5c. Read the comment docs.

@hiroppy
hiroppyforce-pushed the feature/modify-loglevel branch from cb7e81a to 237cfa1CompareJune 8, 2019 11:57
@hiroppyhiroppy changed the title feat(client): delete none and warning because they are deprecated feat(client): delete none and warning from clientLogLevelJun 8, 2019
Comment threadclient-src/default/utils/log.js Outdated
default:
if (level === 'none' || level === 'warning') {
log.error('[WDS] "none" and "warning" have been deprecated.');
}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Remove this, no need this in next release, it is major release

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

fm, I want to avoid receiving many issues about these changes from everyone.😞

@hiroppyhiroppyJun 8, 2019

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

ok, I'll delete here but I'll add warn to master maybe. what do you think?

@alexander-akaitalexander-akait left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good work, one note!

@hiroppy
hiroppyforce-pushed the feature/modify-loglevel branch from 237cfa1 to c226f5cCompareJune 8, 2019 17:44
@hiroppy
hiroppy merged commit 97fd275 into nextJun 10, 2019
@hiroppy
hiroppy deleted the feature/modify-loglevel branch June 10, 2019 14:48
knagaitsev pushed a commit to knagaitsev/webpack-dev-server that referenced this pull request Jul 31, 2019
Sign up for freeto 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.

2 participants

@hiroppy@alexander-akait