Skip to content

add support for amd options #663

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

Open
hanai wants to merge 5 commits into
base: master
Choose a base branch
from
+14 −0
Open

add support for amd options #663

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
fb715a9
add support for amd options
hanai Jun 18, 2020
b84e595
Add options to CLI args list
developit Jun 19, 2020
da36404
Update README.md
hanai Jun 20, 2020
cf7a2f3
Update src/index.js
hanai Jun 20, 2020
69628af
fix read amd options
hanai Jun 20, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
fix read amd options
  • Loading branch information
@hanai
hanai committed Jun 20, 2020
commit 69628af62ec97f6b86e147d1e3a1ad87cb729b77
8 changes: 4 additions & 4 deletions src/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -458,12 +458,12 @@ function createConfig(options, entry, format, writeMeta) {
}

let amd = {};
if (options.amdName) {
amd.id = options.amdName;
if (options['amd-name']) {
amd.id = options['amd-name'];
}

if (options.amdDefine) {
amd.define = options.amdDefine;
if (options['amd-define']) {
amd.define = options['amd-define'];
}
Comment thread
hanai marked this conversation as resolved.

const moduleAliases = options.alias
Expand Down