Skip to content
This repository has been archived by the owner on Aug 31, 2023. It is now read-only.

fix(rome_js_semantic): support for object and array bindings when exporting #3620

Merged
merged 7 commits into from
Nov 10, 2022

Conversation

xunilrj
Copy link
Contributor

@xunilrj xunilrj commented Nov 9, 2022

Summary

This PR correctly identifies object and array bindings when they are being exported:

export const {A} = a;
export const [B] = b;

Test Plan

> cargo test -p rome_js_analyze -- no_unused_variables

@xunilrj xunilrj requested a review from leops as a code owner November 9, 2022 13:28
@netlify
Copy link

netlify bot commented Nov 9, 2022

Deploy Preview for docs-rometools ready!

Name Link
🔨 Latest commit b5e7690
🔍 Latest deploy log https://app.netlify.com/sites/docs-rometools/deploys/636c04b228b0880009dddc06
😎 Deploy Preview https://deploy-preview-3620--docs-rometools.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@xunilrj xunilrj temporarily deployed to netlify-playground November 9, 2022 13:28 Inactive
@xunilrj xunilrj temporarily deployed to netlify-playground November 9, 2022 13:32 Inactive
@github-actions
Copy link

github-actions bot commented Nov 9, 2022

@leops leops added this to the 10.0.1 milestone Nov 9, 2022
@xunilrj xunilrj temporarily deployed to netlify-playground November 9, 2022 17:00 Inactive
@xunilrj xunilrj requested a review from a team November 9, 2022 17:23
@xunilrj xunilrj temporarily deployed to netlify-playground November 9, 2022 17:23 Inactive
@xunilrj xunilrj temporarily deployed to netlify-playground November 9, 2022 18:13 Inactive
@xunilrj xunilrj temporarily deployed to netlify-playground November 9, 2022 18:15 Inactive
@xunilrj xunilrj temporarily deployed to netlify-playground November 9, 2022 19:51 Inactive
@xunilrj xunilrj merged commit 7d9c69c into main Nov 10, 2022
@xunilrj xunilrj deleted the fix/noUnusedVariable-export-with-binding branch November 10, 2022 10:33
jeysal added a commit to jeysal/rometools that referenced this pull request Nov 10, 2022
* upstream/main:
  fix(npm/js-api): Import type from @rometools/backend-jsonrpc (rome#3647)
  doc(npm/js-api): Add experimental warning to README
  fix(npm/js-api): Lazy load backend implementations (rome#3652)
  feat(rome_lsp): stop the daemon after the last client disconnects (rome#3643)
  fix(npm/js_api): Ensure JS API build contains `dist` folder (rome#3648)
  fix(rome_cli): ensures the service only connects to compatible versions (rome#3642)
  feat(playground): add settings button and group IR (rome#3559)
  release: v10.0.1 (rome#3649)
  fix(rome_js_analyze): False positives for interactive elements in `useKeyWithClickEvents` (rome#3644)
  fix(rome_js_semantic): support for object and array bindings when exporting (rome#3620)
  doc(editors): Clarify Rome discovery (rome#3639)
  fix(rome_js_analyze): improve the detection of `ReactDOM.render` calls in `noRenderReturnValue` (rome#3626)
  chore(npm): add license (rome#3629)
  Add rustdocs index
  Add environment
  Change rust docs to use Netlify
  fix(rome_js_analyze): useValidAnchor considering all possible expressions (rome#3599)
  fix(rome_js_formatter): Trailing comma inside import rome#3600 (rome#3624)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants