Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

simplify npm release process #4479

Closed
wants to merge 31 commits into from
Closed

simplify npm release process #4479

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
31 commits
Select commit Hold shift + click to select a range
e1147dc
move src/library.*.js -> npm/*/src/index.js
derhuerst Feb 7, 2017
76dc277
simplify scripts/build-rpc-json.js
derhuerst Feb 7, 2017
9e298b0
simplify scripts/dryrun-npm.sh & scripts/release.sh
derhuerst Feb 7, 2017
e2f4bff
basic smoke tests for modules
derhuerst Feb 7, 2017
2ee8948
bugfixes :bug:
derhuerst Feb 7, 2017
b62e3d9
run module tests on release
derhuerst Feb 7, 2017
68d5917
cleanup, bugfix :bug:
derhuerst Feb 7, 2017
4c6f20d
modules: jsnext:main field, LICENSE file
derhuerst Feb 7, 2017
d781e0e
fix npm script, use relative paths
derhuerst Feb 8, 2017
ef7081f
fix copy scripts
derhuerst Feb 8, 2017
bb98d5b
fix library entry points
derhuerst Feb 8, 2017
5500e21
dryrun-npm -> prepare-npm-libraries, reuse it in release.sh
derhuerst Feb 8, 2017
ffd960c
copy abi/util/address -> 3rdparty/etherscan/util
derhuerst Feb 8, 2017
de56706
polyfills as peer dependencies
derhuerst Feb 8, 2017
5c63681
add dependencies, update readmes
derhuerst Feb 8, 2017
d648f82
Add `mkdir -p` to make sure the directories exist
ngotchac Feb 8, 2017
4173f1b
Fixed tests // Fixed published package.json
ngotchac Feb 8, 2017
27f041e
Create symlink in Etherscan for DRY-love
ngotchac Feb 9, 2017
82866f1
merge master into jr-simplify-release
derhuerst Feb 9, 2017
0760a85
export correctly under ES5, remove unnecessary deps
derhuerst Feb 9, 2017
5da69d7
parity.js -> parity.npm.js, library.parity.js -> parity.js
derhuerst Feb 13, 2017
4b6657d
etherscan: import address util from api
derhuerst Feb 13, 2017
fb1cdc1
Merge branch 'master' into jr-simplify-release
ngotchac Feb 15, 2017
3574107
copy license file to libs
derhuerst Feb 16, 2017
9a478f0
build jsonrpc without redirect
derhuerst Feb 16, 2017
0465034
move lib tests to test/e2e
derhuerst Feb 16, 2017
a82fdbc
use mocha for lib tests
derhuerst Feb 16, 2017
7642710
merge master into jr-simplify-release
derhuerst Feb 21, 2017
d55d051
npm libs: babel-polyfill -> babel-runtime
derhuerst Feb 21, 2017
5c9a018
npm libs: update docs, ignore LICENSE files
derhuerst Feb 21, 2017
d8e282e
npm libs: babel-runtime as non-peer dependency
derhuerst Feb 21, 2017
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
2 changes: 1 addition & 1 deletion js/src/3rdparty/etherscan/util/address.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1 @@
../../../abi/util/address.js
../../../api/util/address.js
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should probably be removed now?

1 change: 1 addition & 0 deletions js/src/api/util/address.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
../../abi/util/address.js
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not moving the file in api ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ngotchac Maybe i missed something, but then abi would depend on api, which we try to prevent.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure I fully followed the discussion about what goes in ABI vs. API. However it just looks weird to have a symlink to symlink.

Copy link
Contributor

@jacogr jacogr Feb 15, 2017 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

~/abi/util is not part of the exposed interfaces (i.e. stuff on Abi.*), so don't use it. :)

  1. Has anybody tested the symlink stuff on Windows? (Anything needs to work across Mac, Linux & Windows - although we may get away here since publishing is only done from Linux)

  2. And I do agree, symlink to symlink is (very) weird, but (slightly) better than the alternative at least.

2 changes: 1 addition & 1 deletion js/src/api/util/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
// You should have received a copy of the GNU General Public License
// along with Parity. If not, see <http://www.gnu.org/licenses/>.

import { isAddress as isAddressValid, toChecksumAddress } from '../../abi/util/address';
import { isAddress as isAddressValid, toChecksumAddress } from './address';
import { abiDecode, decodeCallData, decodeMethodInput, methodToAbi } from './decode';
import { abiEncode, encodeMethodCallAbi } from './encode';
import { bytesToHex, hexToAscii, asciiToHex } from './format';
Expand Down