Skip to content
This repository has been archived by the owner on Jul 9, 2021. It is now read-only.

Change all instances of networkId to chainId #2313

Merged
merged 39 commits into from
Nov 6, 2019
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
39 commits
Select commit Hold shift + click to select a range
119cc57
.gitignore migrations/0x_ganache_snapshot
feuGeneA Oct 16, 2019
ed7c6bc
.gitignore new-ish Python contract wrappers
feuGeneA Oct 16, 2019
4206e6d
rm superfluous contract artifact in Python package
feuGeneA Oct 16, 2019
363363a
Eliminate circular dependency
feuGeneA Oct 18, 2019
7b5bf31
Improve output of monorepo-level parallel script
feuGeneA Oct 22, 2019
923edcd
Silence new versions of linters
feuGeneA Oct 23, 2019
1c1fc50
Support Rich Reverts via Web3.py middleware
feuGeneA Oct 28, 2019
66ce2c0
Fix bug in generated wrappers' bytes handling
feuGeneA Oct 28, 2019
606ec21
Migrate to Exchange v3
feuGeneA Oct 28, 2019
76e8f17
Fix typo in DevUtils documentation
feuGeneA Oct 28, 2019
d702890
Include new contracts in docs
feuGeneA Oct 28, 2019
a04cec5
Re-enable Python checks in CI
feuGeneA Oct 28, 2019
4b02dcc
Accept strings for bytes
feuGeneA Oct 19, 2019
3a8d2a6
Fix CircleCI build artifacts for gen'd python
feuGeneA Oct 23, 2019
348a718
Accept a provider OR a Web3 object
feuGeneA Oct 23, 2019
b2fbbe7
wrapper base: don't assume there are accounts
feuGeneA Oct 25, 2019
ea77609
Eliminate some inline linter directives
feuGeneA Oct 27, 2019
6eb5ca0
make CHANGELOGs be REVERSE chronological
feuGeneA Oct 28, 2019
bc354e3
Update CHANGELOG entries and bump version numbers
feuGeneA Oct 29, 2019
4667fc5
Merge branch 'development' into feat/3.0/python
feuGeneA Oct 29, 2019
0ef3435
@0x/contract-addresses: Put addr's in JSON, not TS
feuGeneA Oct 29, 2019
6010a0f
Merge branch 'development' into feat/3.0/python
feuGeneA Nov 2, 2019
5873b56
sra_client.py: incl. docker in `./setup.py clean`
feuGeneA Oct 19, 2019
8941145
sra_client.py: Migrate to protocol v3
feuGeneA Oct 21, 2019
1ec5098
abi-gen/test: recompile contract fixtures for 3.0
feuGeneA Nov 2, 2019
510e341
Sync `monorepo$ yarn test` exclusions to CI config
feuGeneA Nov 1, 2019
23b8009
utils: fix typo in comment
feuGeneA Nov 2, 2019
6064a2b
sra-spec: correct typo
feuGeneA Nov 1, 2019
de1b395
contract-wrappers: TODO after coord.-server update
feuGeneA Nov 1, 2019
629af13
Refactor networkId to chainId everywhere
feuGeneA Nov 1, 2019
8675663
abi-gen/templates/Py: clarify if/else logic
feuGeneA Nov 5, 2019
e472cee
sra_client.py: Update CHANGELOG and bump version
feuGeneA Nov 5, 2019
8d7f47f
contract_addresses/setup.py: rm unnecessary rm
feuGeneA Nov 5, 2019
6d454d1
json_schemas.py: corrections to dev dependencies
feuGeneA Nov 5, 2019
a14a4d3
In tests against deployment, also run doctests
feuGeneA Oct 29, 2019
19fac92
contract_wrappers example: rm xtra Order attribute
feuGeneA Nov 5, 2019
004fcd9
Update CHANGELOGs
feuGeneA Nov 5, 2019
3cfd6fe
Merge branch 'feat/3.0/python' into chain-id-not-network-id
feuGeneA Nov 5, 2019
d9a409c
Merge branch 'development' into chain-id-not-network-id
feuGeneA Nov 6, 2019
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
10 changes: 9 additions & 1 deletion .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,9 @@ jobs:
keys:
- repo-{{ .Environment.CIRCLE_SHA1 }}
- run: yarn wsrun test:circleci @0x/contracts-multisig @0x/contracts-utils @0x/contracts-exchange-libs @0x/contracts-erc20 @0x/contracts-erc721 @0x/contracts-erc1155 @0x/contracts-asset-proxy @0x/contracts-exchange-forwarder @0x/contracts-tests @0x/contracts-staking @0x/contracts-coordinator
# TODO(dorothy-zbornak): Re-enable after updating this package for 3.0.
# TODO(dorothy-zbornak): Re-enable after updating this package for
# 3.0. At that time, also remove exclusion from monorepo
# package.json's test script.
# - run: yarn wsrun test:circleci @0x/contracts-extensions
test-publish:
resource_class: medium+
Expand Down Expand Up @@ -116,6 +118,9 @@ jobs:
- run: yarn wsrun test:circleci @0x/abi-gen
# TODO (xianny): Needs to be updated for 3.0
# - run: yarn wsrun test:circleci @0x/asset-buyer
# TODO: Needs to be updated for 3.0. At that time, also remove
# exclusion from monorepo package.json's test script.
# - run: yarn wsrun test:circleci @0x/asset-swapper
- run: yarn wsrun test:circleci @0x/contract-artifacts
- run: yarn wsrun test:circleci @0x/assert
- run: yarn wsrun test:circleci @0x/base-contract
Expand All @@ -125,6 +130,9 @@ jobs:
- run: yarn wsrun test:circleci @0x/dev-utils
- run: yarn wsrun test:circleci @0x/json-schemas
- run: yarn wsrun test:circleci @0x/order-utils
# TODO: Needs to be updated for 3.0. At that time, also remove
# exclusion from monorepo package.json's test script.
# - run: yarn wsrun test:circleci @0x/orderbook
- run: yarn wsrun test:circleci @0x/sol-compiler
- run: yarn wsrun test:circleci @0x/sol-tracing-utils
- run: yarn wsrun test:circleci @0x/sol-doc
Expand Down
1 change: 0 additions & 1 deletion contracts/test-utils/src/constants.ts
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ const MAX_UINT256 = new BigNumber(2).pow(256).minus(1);
export const constants = {
BASE_16: 16,
INVALID_OPCODE: 'invalid opcode',
TESTRPC_NETWORK_ID: 50,
TESTRPC_CHAIN_ID: 1337,
// Note(albrow): In practice V8 and most other engines limit the minimum
// interval for setInterval to 10ms. We still set it to 0 here in order to
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@
"contracts:watch": "wsrun watch $PKG --parallel --exclude-missing",
"remove_node_modules": "lerna clean --yes; rm -rf node_modules",
"rebuild": "run-s clean build",
"test": "wsrun test $PKG --fast-exit --serial --exclude-missing",
"test": "wsrun test $PKG --fast-exit --serial --exclude-missing --exclude @0x/asset-swapper --exclude @0x/orderbook --exclude @0x/contracts-extensions",
"test:contracts": "wsrun test -p ${npm_package_config_contractsPackages} -c --fast-exit --serial --exclude-missing",
"generate_doc": "node ./packages/monorepo-scripts/lib/doc_generate.js",
"upload_md_docs": "aws s3 rm --recursive s3://docs-markdown; wsrun s3:sync_md_docs --exclude-missing",
Expand Down
9 changes: 9 additions & 0 deletions packages/0x.js/CHANGELOG.json
Original file line number Diff line number Diff line change
@@ -1,4 +1,13 @@
[
{
"version": "8.0.0-beta.0",
"changes": [
{
"note": "Exported intefaces changed: from getContractAddressesForNetworkOrThrow to getContractAddressesForChainOrThrow, from NetworkId to ChainId, from ContractNetworks to ContractChains, and from ContractNetworkData to ContractChainData.",
"pr": 2313
}
]
},
{
"version": "7.1.0-beta.0",
"changes": [
Expand Down
6 changes: 3 additions & 3 deletions packages/0x.js/src/index.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
export { getContractAddressesForNetworkOrThrow, NetworkId, ContractAddresses } from '@0x/contract-addresses';
export { getContractAddressesForChainOrThrow, ChainId, ContractAddresses } from '@0x/contract-addresses';

export {
assetDataUtils,
Expand Down Expand Up @@ -124,11 +124,11 @@ export {
BlockParam,
CompilerOpts,
StandardContractOutput,
ContractNetworks,
ContractChains,
TxDataPayable,
BlockParamLiteral,
CompilerSettings,
ContractNetworkData,
ContractChainData,
DevdocOutput,
EvmOutput,
CompilerSettingsMetadata,
Expand Down
4 changes: 4 additions & 0 deletions packages/abi-gen/CHANGELOG.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,10 @@
{
"note": "In Python wrappers, fix bug with casting some bytes objects using bytes.fromhex()",
"pr": 2284
},
{
"note": "Command-line argument network-id has changed to chain-id",
"pr": 2313
}
]
},
Expand Down
8 changes: 4 additions & 4 deletions packages/abi-gen/src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ import { utils } from './utils';
const ABI_TYPE_CONSTRUCTOR = 'constructor';
const ABI_TYPE_METHOD = 'function';
const ABI_TYPE_EVENT = 'event';
const DEFAULT_NETWORK_ID = 50;
const DEFAULT_CHAIN_ID = 1337;
const DEFAULT_BACKEND = 'web3';

const args = yargs
Expand Down Expand Up @@ -65,10 +65,10 @@ const args = yargs
choices: [ContractsBackend.Web3, ContractsBackend.Ethers],
default: DEFAULT_BACKEND,
})
.option('network-id', {
describe: 'ID of the network where contract ABIs are nested in artifacts',
.option('chain-id', {
describe: 'ID of the chain where contract ABIs are nested in artifacts',
type: 'number',
default: DEFAULT_NETWORK_ID,
default: DEFAULT_CHAIN_ID,
})
.option('language', {
describe: 'Language of output file to generate',
Expand Down
Loading