Unverified Commit cc8dca19 authored by yann300's avatar yann300 Committed by GitHub

Merge pull request #1240 from ethereum/bultinf

exclude builtin version from check
parents 8c767694 e144e82e
......@@ -238,8 +238,8 @@ jobs:
steps:
- checkout
- run: npm install
- run: npx nx build remix-ide --with-deps
- run: npm run downloadsolc_assets
- run: npx nx build remix-ide --with-deps
- run:
name: Deploy
command: |
......@@ -267,6 +267,7 @@ jobs:
- checkout
- setup_remote_docker
- run: npm install
- run: npm run downloadsolc_assets
- run: npx nx build remix-ide --with-deps
- run: ./apps/remix-ide/ci/copy_resources.sh
- run: ./apps/remix-ide/ci/publishIpfs
......@@ -291,8 +292,8 @@ jobs:
steps:
- checkout
- run: npm install
- run: npx nx build remix-ide --with-deps
- run: npm run downloadsolc_assets
- run: npx nx build remix-ide --with-deps
- run:
name: Deploy
command: |
......@@ -320,8 +321,8 @@ jobs:
- checkout
- run: npm install
- run: npm run build:libs
- run: npm run build
- run: npm run downloadsolc_assets
- run: npm run build
- run:
name: Deploy
command: |
......
......@@ -170,6 +170,7 @@ class CompilerContainer {
_retrieveVersion (version) {
if (!version) version = this._view.versionSelector.value
if (version === 'builtin') version = this.data.defaultVersion
return semver.coerce(version) ? semver.coerce(version).version : ''
}
......@@ -487,7 +488,7 @@ class CompilerContainer {
this._view.versionSelector.appendChild(option)
}
})
if (semver.lt(this._retrieveVersion(), 'v0.4.12+commit.194ff033.js')) {
if (this.data.selectedVersion !== 'builtin' && semver.lt(this._retrieveVersion(), 'v0.4.12+commit.194ff033.js')) {
toaster(yo`
<div>
<b>Old compiler usage detected.</b>
......@@ -500,7 +501,7 @@ class CompilerContainer {
// Workers cannot load js on "file:"-URLs and we get a
// "Uncaught RangeError: Maximum call stack size exceeded" error on Chromium,
// resort to non-worker version in that case.
if (this.data.selectedVersion !== 'builtin' && canUseWorker(this.data.selectedVersion)) {
if (canUseWorker(this._retrieveVersion())) {
this.compileTabLogic.compiler.loadVersion(true, url)
this.setVersionText('(loading using worker)')
} else {
......
......@@ -50,7 +50,7 @@
"bumpVersion:libs": "gulp & gulp syncLibVersions;",
"browsertest": "sleep 5 && npm run nightwatch_local",
"csslint": "csslint --ignore=order-alphabetical --errors='errors,duplicate-properties,empty-rules' --exclude-list='apps/remix-ide/src/assets/css/font-awesome.min.css' apps/remix-ide/src/assets/css/",
"downloadsolc_assets": "wget --no-check-certificate https://binaries.soliditylang.org/bin/soljson-v0.8.4+commit.c7e474f2.js -O ./apps/remix-ide/src/assets/js/soljson.js",
"downloadsolc_assets": "wget --no-check-certificate https://binaries.soliditylang.org/wasm/soljson-v0.8.4+commit.c7e474f2.js -O ./apps/remix-ide/src/assets/js/soljson.js",
"make-mock-compiler": "node apps/remix-ide/ci/makeMockCompiler.js",
"minify": "uglifyjs --in-source-map inline --source-map-inline -c warnings=false",
"nightwatch_parallel": "npm run build:e2e && nightwatch --config dist/apps/remix-ide-e2e/nightwatch.js --env=chrome,firefox",
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment