Skip to content
This repository has been archived by the owner on Jan 18, 2023. It is now read-only.

Zero ex wrapper 2 #141

Merged
merged 4 commits into from
Aug 1, 2018
Merged

Zero ex wrapper 2 #141

merged 4 commits into from
Aug 1, 2018

Conversation

asoong
Copy link
Contributor

@asoong asoong commented Jul 31, 2018

No description provided.

@asoong asoong requested a review from a team July 31, 2018 23:28
@coveralls
Copy link

coveralls commented Jul 31, 2018

Pull Request Test Coverage Report for Build 959

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 950: 0.0%
Covered Lines: 282
Relevant Lines: 282

💛 - Coveralls

address[] memory takerTokens = new address[](_orderCount);
uint256[] memory takerAmounts = new uint256[](_orderCount);

// First 32 bytes are reseved for the number of orders
Copy link
Contributor

Choose a reason for hiding this comment

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

"Reserved"

expect(parsedMakerAssetData).to.equal(makerAssetData);
});

it('should correctly parse the taker asset data', async () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Dang, this is what you meant by separate tests

@@ -3,7 +3,7 @@
"outDir": "./transpiled/",
"sourceMap": true,
"noImplicitAny": true,
"lib": ["es2015", "es2016", "es2017"],
"lib": ["dom", "es2015", "es2016", "es2017"],
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need this?

@asoong asoong merged commit 5b499d3 into master Aug 1, 2018
@asoong asoong deleted the zero-ex-wrapper-2 branch August 1, 2018 01:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants