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.

Sign up for GitHub

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

Jump to bottom

fix: remove referenced identifiers #949

Merged
merged 1 commit into from
Aug 26, 2021
Merged

fix: remove referenced identifiers #949

merged 1 commit into from
Aug 26, 2021

Conversation

luhc228
Copy link
Contributor

@luhc228 luhc228 commented Aug 26, 2021 •
edited
Loading

背景

对于以下两种情况，如果 b 没有被引用，会把整个语句删掉：

const [a, b] = [1, 2];
const {a, b} = obj;

期望是只把 b 删掉就行了

@luhc228 luhc228 self-assigned this Aug 26, 2021
@fyangstudio fyangstudio merged commit 21a810a into release/1.4.1 Aug 26, 2021
@fyangstudio fyangstudio deleted the fix/remove-referenced-identifiers branch August 26, 2021 02:52
@fyangstudio fyangstudio mentioned this pull request Aug 26, 2021
Merged
luhc228 added a commit that referenced this pull request Aug 26, 2021
@fyangstudio @luhc228 @GiveMe-A-Name
Release 1.4.1 (#943)
83954c4 
* chore: remove @ali/publish-visual extension (#938)

* chore: remove @ali/publish-visual extension

* fix: material-help (#944)

* fix: material-help (#946)

* fix: material-help

* chore: add @ali/publish-visual to o2 pack (#948)

* fix: import duplicate components when add routers (#947)

* fix: import duplicate components when add routers

* chore: remove comment

* feat: update feedbackLink (#940)

* feat: remove deprecated extension (#939)

* fix: fail to remove compoennt (#942)

Co-authored-by: yangfan <18767120422@163.com>

* feat: add extensionDependencies (#945)

* feat: update version and changelog (#941)

* feat: update version and changelog

* feat: update changelog

* feat: update

* fix: remove referenced identifiers (#949)

Co-authored-by: Hengchang Lu <44047106+luhc228@users.noreply.github.com>
Co-authored-by: qixuan <58852732+GiveMe-A-Name@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fyangstudio fyangstudio fyangstudio approved these changes

Assignees

@luhc228 luhc228

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@luhc228 @fyangstudio