Skip to content

expose getLocalIdent #357

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

Merged
merged 2 commits into from
Nov 18, 2016
Merged

expose getLocalIdent #357

merged 2 commits into from
Nov 18, 2016

Conversation

longlho
Copy link
Contributor

@longlho longlho commented Oct 10, 2016

fixes #292

@codecov-io
Copy link

codecov-io commented Oct 10, 2016

Current coverage is 98.35% (diff: 100%)

Merging #357 into master will increase coverage by <.01%

@@             master       #357   diff @@
==========================================
  Files             9          9          
  Lines           303        304     +1   
  Methods          41         41          
  Messages          0          0          
  Branches         66         67     +1   
==========================================
+ Hits            298        299     +1   
  Misses            5          5          
  Partials          0          0          

Powered by Codecov. Last update 9df4467...b54c4f6

@longlho longlho changed the title expose generateScopedName expose getLocalIdent Oct 10, 2016
@sokra
Copy link
Member

sokra commented Nov 17, 2016

Without the module loading stuff it sounds doable. Just pass getLocalId via option/query. webpack 2 accepts non-JSON options.

@MrCheater
Copy link

Add an example of usage to documentation, please.

@longlho
Copy link
Contributor Author

longlho commented Nov 17, 2016

@sokra gotcha I'll remove that

@jsf-clabot
Copy link

jsf-clabot commented Nov 17, 2016

CLA assistant check
All committers have signed the CLA.

@longlho
Copy link
Contributor Author

longlho commented Nov 17, 2016

@sokra @MrCheater done

@longlho
Copy link
Contributor Author

longlho commented Nov 18, 2016

can we merge & release :)?

@sokra sokra merged commit 46ebca3 into webpack-contrib:master Nov 18, 2016
@esquevin
Copy link

esquevin commented Dec 2, 2016

Any idea when this will be released ?

@bebraw
Copy link
Contributor

bebraw commented Dec 5, 2016

@esquevin Looks like it's included in 0.26.1 released three days ago.

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.

Custom localIdent name as a function
7 participants