Skip to content

feat: emit const instead of let for CommonJS require#1070

Merged
anmonteiro merged 4 commits intomainfrom
anmonteiro/emit-const-for-requires
Feb 17, 2024
Merged

feat: emit const instead of let for CommonJS require#1070
anmonteiro merged 4 commits intomainfrom
anmonteiro/emit-const-for-requires

Conversation

@anmonteiro
Copy link
Copy Markdown
Member

follow-up to #1019 and #1059

  • emits const x = require('x') instead of let x = require ..
  • melange never mutates the require assignment, so this is yet another place where we can emit const instead.

@anmonteiro anmonteiro merged commit 1085376 into main Feb 17, 2024
@anmonteiro anmonteiro deleted the anmonteiro/emit-const-for-requires branch February 17, 2024 03:10
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.

1 participant