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

Make getBinding ignore labels; add Scope#getLabel, Scope#registerLabel #4758

Merged
merged 3 commits into from Oct 21, 2016

Conversation

kangax
Copy link
Member

@kangax kangax commented Oct 20, 2016

Q A
Bug fix? no
Breaking change? sort of
New feature? sort of
Deprecations? sort of
Spec compliancy? n/a
Tests added/pass? yes

Instead of #3649

@codecov-io
Copy link

codecov-io commented Oct 21, 2016

Current coverage is 88.82% (diff: 100%)

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

@@             master      #4758   diff @@
==========================================
  Files           196        196          
  Lines         13868      13875     +7   
  Methods        1430       1432     +2   
  Messages          0          0          
  Branches       3198       3198          
==========================================
+ Hits          12317      12324     +7   
  Misses         1551       1551          
  Partials          0          0          

Powered by Codecov. Last update 7443f9e...10d37b2

@hzoo hzoo added the PR: New Feature 🚀 A type of pull request used for our changelog categories label Oct 21, 2016
@danez
Copy link
Member

danez commented Oct 21, 2016

What about the comment in #3649 from @kittens ? Is this still valid?

@hzoo
Copy link
Member

hzoo commented Oct 21, 2016

Ok added/created a hasLabel check and a test (it was generating the same name before so it was a valid concern)

@hzoo hzoo merged commit beda884 into master Oct 21, 2016
mstade added a commit to zambezi/babel that referenced this pull request Oct 28, 2016
* master: (38 commits)
  chore(package): update browserify to version 13.1.1 (babel#4762)
  Increase test coverage (babel#4742)
  Make getBinding ignore labels; add Scope#getLabel, Scope#registerLabel (babel#4758)
  Add variance node type and generate property variance annotations (babel#4697)
  Add make command to delete node_modules (babel#4748)
  fixes [skip ci]
  Support ObjectExpression in static path evaluation (babel#4746)
  Fix replacing for-of if inside label (babel#4736)
  Replace `path-exists` with `fs.existsSync` (babel#4731)
  Avoid unnecessary +0 in transform-es2015-parameters (babel#4738)
  [import()] Initial support for dynamic-import (babel#4699)
  Fix line endings on checkout
  Automatically generate missing expected.js fixtures (babel#4735)
  Fix few typos in issue/pr templates (babel#4739) [skip ci]
  contributing updates [skip ci]
  increase git depth [skip ci]
  Change usage of "suite"/"test" in unit-tests to "describe"/"it" (babel#4734)
  Run ESLint on test files, and fix lint errors in test files (babel#4732)
  Add .gitattributes forcing LF line endings (babel#4730)
  Update tests for changed error messages in Babylon (babel#4727)
  ...
@hzoo hzoo deleted the labels branch November 1, 2016 21:03
panagosg7 pushed a commit to panagosg7/babel that referenced this pull request Jan 17, 2017
babel#4758)

* Make getBinding ignore labels; add Scope#getLabel, Scope#registerLabel

* generateUid: account for labels again
@lock lock bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Oct 7, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Oct 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated A closed issue/PR that is archived due to age. Recommended to make a new issue PR: New Feature 🚀 A type of pull request used for our changelog categories
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants