Require reason message removed
Inline documentation in NatSpec added
Snapshot added to _mint and _burn method

…lin#1329)

* release candidate v2.0.0-rc.1

* fix linter error

(cherry picked from commit c12a1c6)

* Roles now emit events in construction and when renouncing.

* Add the missing test for ERC721Holder

* fix lint

* Move the holder test to a separate file

* Removed mintingFinished from ERC20Mintable.

* Removed MintingFinished from ERC721Mintable.

* Removed MintingFinished event.

* Improved bounty tests.

* Fixed linter errors.

* Addressed review comments.

…Zeppelin#1338)

* Add BigNumber support to expectEvent/inLogs (OpenZeppelin#1026)

* switched direct logs array check to expectEvent method in AllowanceCrowdsale.test.js

* Refactor expectEvent.inLogs function to use simple value number check

* Introduced should.be.bignumber method to compare BigNumber values

* Destructure transaction object to extract logs field

* fix: refactor sign.js and related tests

* fix: remove unused dep

* fix: update package.json correctly

* Add missing tests to ECRecovery

* fix lint

* Reorganize the tests

* Reuse signature

* fix static errors

* Apply suggestions by @frangion and @nventuro

* Remove only

* More suggestions

* Remove unnecessary max-len

* remove only

…penZeppelin#1343)

* Add BigNumber support to expectEvent/inLogs (OpenZeppelin#1026)

* switched direct logs array check to expectEvent method in AllowanceCrowdsale.test.js

* Refactor expectEvent.inLogs function to use simple value number check

* Introduced should.be.bignumber method to compare BigNumber values

* Use expectEvent to test logs (OpenZeppelin#1232)

* Removed trailing space

…n#1360)

* add test for msg.data too short

* fix test to hit that branch

* Update SignatureBouncer.test.js

* signing prefix added

* Minor improvement

* Tests changed

* Successfully tested

* Minor improvements

* Minor improvements

* Revert "Dangling commas are now required. (OpenZeppelin#1359)"

This reverts commit a688977.

* fixes OpenZeppelin#1358

* linting done

* suggested changes

…ppelin#1368)

* signing prefix added

* Minor improvement

* Tests changed

* Successfully tested

* Minor improvements

* Minor improvements

* Revert "Dangling commas are now required. (OpenZeppelin#1359)"

This reverts commit a688977.

* fixex OpenZeppelin#1355

* linting

* suggested changes

* Update BreakInvariantBounty.test.js

…nZeppelin#1364)

* signing prefix added

* Minor improvement

* Tests changed

* Successfully tested

* Minor improvements

* Minor improvements

* Revert "Dangling commas are now required. (OpenZeppelin#1359)"

This reverts commit a688977.

* updates

* fixes OpenZeppelin#1371

* Removed extra whitespace

* signing prefix added

* Minor improvement

* Tests changed

* Successfully tested

* Minor improvements

* Minor improvements

* Revert "Dangling commas are now required. (OpenZeppelin#1359)"

This reverts commit a688977.

* updates

* fixes OpenZeppelin#1356

* Removed extra semicolon.

* Update issue templates

* improve pull request template

* remove previous issue template

)

* add note about compiling artifacts to release notes

* add explanation of truffle bug

* fixes OpenZeppelin#1372

* done in ERC20Capped and ERC20Mintable

* signing prefix added

* Minor improvement

* Tests changed

* Successfully tested

* Minor improvements

* Minor improvements

* Revert "Dangling commas are now required. (OpenZeppelin#1359)"

This reverts commit a688977.

* updates

* fixes OpenZeppelin#1200

* suggested change

* separate part of ERC721Mintable into ERC721MetadataMintable

* remove mint and burn from 721 tests

* Fixed linter error.

* fix ERC721 mint tests

* Minor fixes.

…pelin#1381)

* signing prefix added

* Minor improvement

* Tests changed

* Successfully tested

* Minor improvements

* Minor improvements

* Revert "Dangling commas are now required. (OpenZeppelin#1359)"

This reverts commit a688977.

* updates

* fixes OpenZeppelin#1384

* introduced claimable and cancelBounty

* cancelBounty tests

* Update BreakInvariantBounty.test.js

…1375)

*     Add Arrays library with unit tests (OpenZeppelin#1209)

    * prepared due to snapshot token requirements
    * add library with method to find upper bound
    * add unit test for basic and edge cases

* Imporove documentation for Arrays library

Simplify Arrays.test.js to use short arrays as test date

* Added comment for uint256 mid variable.
* Explaned why uint256 mid variable calculated as Math.average is safe to use as index of array.

Require reason message removed
Inline documentation in NatSpec added
Snapshot added to _mint and _burn method

….com/jbogacz/openzeppelin-solidity into fix/ERC20-token-with-snapshots-#1209