Skip to content

Comments

chore: deprecate object syntax for proxy#3224

Closed
rishabh3112 wants to merge 4 commits intowebpack:masterfrom
rishabh3112:chore/deprecate-proxy-object
Closed

chore: deprecate object syntax for proxy#3224
rishabh3112 wants to merge 4 commits intowebpack:masterfrom
rishabh3112:chore/deprecate-proxy-object

Conversation

@rishabh3112
Copy link
Member

  • 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?

Will do if required.

Motivation / Use-Case

#2454
#2454 (comment)

Breaking Changes

Additional Info

@codecov
Copy link

codecov bot commented Apr 24, 2021

Codecov Report

Merging #3224 (a5c93c5) into master (1997a6e) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3224   +/-   ##
=======================================
  Coverage   95.19%   95.19%           
=======================================
  Files          37       37           
  Lines        1248     1249    +1     
  Branches      352      352           
=======================================
+ Hits         1188     1189    +1     
  Misses         54       54           
  Partials        6        6           
Impacted Files Coverage Δ
lib/Server.js 94.56% <100.00%> (+0.01%) ⬆️

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 1997a6e...a5c93c5. Read the comment docs.

@snitin315 snitin315 mentioned this pull request May 28, 2021
6 tasks
Copy link
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

Sorry for delay and fixed in master and in 5.0.0 version

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.

3 participants