Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 48ce92e

Browse filesBrowse files
authored
Merge pull request plotly#6437 from plotly/fix-const-error
Fix es5 dist on master
2 parents 73f9691 + 30d0a99 commit 48ce92e
Copy full SHA for 48ce92e

File tree

Expand file treeCollapse file tree

2 files changed

+4
-7
lines changed
Filter options
Expand file treeCollapse file tree

2 files changed

+4
-7
lines changed

‎.circleci/config.yml

Copy file name to clipboardExpand all lines: .circleci/config.yml
+3-6Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -403,15 +403,12 @@ jobs:
403403
- run:
404404
name: Test plotly.min.js import using requirejs
405405
command: npm run test-requirejs
406-
- run:
407-
name: Display function constructors in all bundles (only on master)
408-
command: |
409-
if [ $CIRCLE_BRANCH == "master" ]
410-
then npm run no-new-func
411-
fi
412406
- run:
413407
name: Test plotly bundles against es6
414408
command: npm run no-es6-dist
409+
- run:
410+
name: Display function constructors in all bundles
411+
command: npm run no-new-func
415412

416413
workflows:
417414
version: 2

‎webpack.config.js

Copy file name to clipboardExpand all lines: webpack.config.js
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ module.exports = {
2222
}
2323
}, {
2424
test: /\.js$/,
25-
include: /node_modules[\\\/]buffer[\\\/]/,
25+
include: /node_modules[\\\/](buffer|is-mobile)[\\\/]/,
2626
use: {
2727
loader: 'babel-loader',
2828
options: {

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.