{"push_id":2722126812,"size":2,"distinct_size":1,"ref":"refs/heads/remove-generatewithstratis","head":"1b9752040b98d90ffe5d39cb29a35bad149437fb","before":"b79c044e55de0e5c384bb7354e1d327c715a05aa","commits":[{"sha":"bf7c94d0d9959a6be79910ff072d7e805f9fb01e","author":{"name":"Francois de la Rouviere","email":"6f4bbf3c79085089f01a384adecf2e945e3a4f92@gmail.com"},"message":"Merge Smart Contracts Master from VSTS (#1639)\n\n* Don't serialize nulls\r\n\r\n* Added minting function to standard token\r\n\r\n* FIxed if conditions\r\n\r\n* fixed address check\r\n\r\n* Update VSIX\r\n\r\n* Fix method to invoke name is null\r\n\r\n* Fix to stop double logging exceptions\r\n\r\n* Handle unhandled exceptions in the executor\r\n\r\n* Better logging\r\n\r\n* Added more logging in the CondensingTx class\r\n\r\n* Update CondensingTx logs\r\n\r\n* Merged PR 235: Fixing mining breaking when a contract would end up with a balance of 0 after a transaction.\r\n\r\n* Merged PR 236: NuGet versions for Stratis.SmartContracts\r\n\r\nSet the version to 0.1.0-alpha for the initial release, according to semver guidelines.\r\n\r\n`Major version zero (0.y.z) is for initial development. Anything may change at any time. The public API should not be considered stable.`\r\n\r\nFor further changes, we can progress like this:\r\n0.2.0-alpha < 0.3.0-alpha < 0.4.0-beta < 1.0.0-rc.1 < 1.0.0 < 2.0.0 < 3.0.0 etc.\r\n\r\n* Merged PR 237: Move config to Stratis folder\r\n\r\nMove config to Stratis folder\r\n\r\n* Merged PR 239: Fix template changes that broke the build 🚀\r\n\r\n* Handle unhandled exception\r\n\r\n* Removed \"allowed\" functionality\r\n\r\n* Fix feeAmount format\r\n\r\n* Improve error\r\n\r\n* Merged PR 243: NodeSettings set to new UserAgent\r\n\r\nStill need to adjust version somehow.\r\n\r\n* Merged PR 245: Updates to SCT / Fix constructor validation\r\n\r\n* Merged PR 246: Update RLP encoding to account for little endianness\r\n\r\nRLP Encoding Tests\r\n\r\n* Deployer text\r\n\r\n* Test Fixes\r\n\r\n* Redundant line...\r\n\r\n* Merged PR 247: Remove coinbase maturity from SmartContractsController\r\n\r\nRemove coinbase maturity from SmartContractsController\r\n\r\n* Fixed issue with request logger\r\n\r\n* Update readme\r\n\r\n* Bump version to v0.8.1\r\n\r\n* Stratis.PatriciaTrie is building\r\n\r\n* PatriciaTrie new project\r\n\r\n* Docs\r\n\r\n* Docs\r\n\r\n* Small code style improvements\r\n\r\n* API Updates\r\n\r\n* StratisPatricia NuGet\r\n\r\n* Patricia moved to NuGet\r\n\r\n* Add validation warnings for fields used at a class level\r\n\r\n* Nuget update :)\r\n\r\n* Unused variable\r\n\r\n* Referenes put back\r\n\r\n* Update field validation text\r\n\r\n* Remove word\r\n\r\n* Change warn format to be in line with error format\r\n\r\n* Remove unused test\r\n\r\n* Add support for long in method params\r\n\r\n* Stratis.Patricia ready\r\n\r\n* Merged PR 250: Merge latest code from Full Node master\r\n\r\nA lot of changes to NBitcoin / Block Assemblers and the like.\r\n\r\n* Revert \"Merged PR 251: Add support for long in method params\r\n\r\nAdd support for long in method params\r\n\r\nRelated work items: #1444\"\r\n\r\n* Bump Version to v0.8.2\r\n\r\n* Fix bug preventing transactions receiving mempool in certain circumstances\r\n\r\n* Update readme with release notes for v0.8.2\r\n\r\n* Merged PR 255: Add support for long in method params\r\n\r\nAdd support for long in method params\r\n\r\nRelated work items: #1444\r\n\r\n* Add base solution\r\n\r\n* Separate interfaces\r\n\r\n* Rename validation error for now\r\n\r\n* Move validation error\r\n\r\n* Move interfaces\r\n\r\n* Separate into classes\r\n\r\n* Move determinism validators\r\n\r\n* Move most format validators\r\n\r\n* Amending to gas prices\r\n\r\n* Rename to IValidator\r\n\r\n* Rename to ValidationResult\r\n\r\n* Remove unused field\r\n\r\n* Move ValidationResult\r\n\r\n* Refactor IValidator validate signature\r\n\r\n* Move IValidator\r\n\r\n* Tests fixed\r\n\r\n* Separate out SmartContract type def validation\r\n\r\n* Move IValidator back\r\n\r\n* Fix integration tests\r\n\r\n* comment\r\n\r\n* Undo IValidator signature change\r\n\r\n* Make SmartContractValidator implement ISmartContractValidator\r\n\r\n* Move ValidationResult\r\n\r\n* Rename to SmartContractValidationResult\r\n\r\n* Rename\r\n\r\n* Rename\r\n\r\n* Make immutable\r\n\r\n* Solution builds\r\n\r\n* Remove MethodFullName\r\n\r\n* Add strongly typed results\r\n\r\n* Merged PR 259: Try Catch Validation\r\n\r\nTry Catch Validation\r\n\r\nRelated work items: #1506\r\n\r\n* Merged PR 256: Stores receipts that can be accessed via API\r\n\r\n* new rule\r\n\r\n* 0.9.0\r\n\r\n* Magic and genesis nonce\r\n\r\n* Merged PR 252: Prevent storage of unsupported Types in PersistentState\r\n\r\nChanges the PersistentState API surface to only expose Types that we can currently serialize.\r\n\r\nRelated work items: #1447\r\n\r\n* Update template version\r\n\r\n* Update NuGet version\r\n\r\n* Remove unused\r\n\r\n* Refactoring\r\n\r\n* csproj update\r\n\r\n* README updates\r\n\r\n* Use IEnumerable\r\n\r\n* Move ModuleValidator\r\n\r\n* Begin rename\r\n\r\n* Finish rename\r\n\r\n* Move SC validation to own project\r\n\r\n* Move to SC Validation project\r\n\r\n* Inject allowed assemblies\r\n\r\n* Remove Stratis.System.Inspector\r\n\r\n* Inject allowed assemblies again\r\n\r\n* Fix missing projects\r\n\r\n* Rule Registration\r\n\r\n* Controller self review\r\n\r\n* Self review\r\n\r\n* Very broken tests but more modular\r\n\r\n* Implementation all fixed - now just for tests\r\n\r\n* Test improvements\r\n\r\n* All normal tests passing\r\n\r\n* Fix merge conflicts\r\n\r\n* Make extension method\r\n\r\n* Handle existing implementation factory\r\n\r\n* Remove unused classes\r\n\r\n* Docs\r\n\r\n* Review\r\n\r\n* Remove file\r\n\r\n* DBreeze Tests\r\n\r\n* Remove old test parameter\r\n\r\n* Renaming\r\n\r\n* Merged PR 272: Merge latest from FullNode\r\n\r\n* Merged PR 276: Merge remote-tracking branch 'upstream/master'\r\n\r\nMerge remote-tracking branch 'upstream/master'\r\n\r\n* Merged PR 277: Merge remote-tracking branch 'upstream/master'\r\n\r\nMerge remote-tracking branch 'upstream/master'\r\n\r\n* in progress\r\n\r\n* Improved struct serialization\r\n\r\n* Improving tests\r\n\r\n* Review\r\n\r\n* Review\r\n\r\n* Consolidate versions\r\n\r\n* Addnode\r\n\r\n* Merged PR 279: Merge remote-tracking branch 'upstream/master'\r\n\r\nMerge remote-tracking branch 'upstream/master'\r\n\r\n* Fix integration test\r\n\r\n* Working on Allowing OP_CREATE\r\n\r\n* Add extra logging to SmartContractCoinviewRule\r\n\r\n* Merged PR 280: Merge latest changes from FN.\r\n\r\n* Local tests passing\r\n\r\n* Integration tests passing\r\n\r\n* Merged PR 285: Merge from FN\r\n\r\n* Merged PR 284: Assert contains message\r\n\r\nAssert contains message\r\n\r\n* Merged PR 282: Clean up SmartContractExecutorTests\r\n\r\n* Merged PR 286: Merge remote-tracking branch 'upstream/master'\r\n\r\nMerge remote-tracking branch 'upstream/master'\r\n\r\n* Fix injection\r\n\r\n* ResultProcessorTests\r\n\r\n* Ready\r\n\r\n* Controller test\r\n\r\n* mempool fix\r\n\r\n* Test\r\n\r\n* Spacing\r\n\r\n* Merged PR 289: Merge remote-tracking branch 'upstream/master'\r\n\r\nMerge remote-tracking branch 'upstream/master'\r\n\r\n* TransactionBuilder adjusted\r\n\r\n* SharedSteps update and docs\r\n\r\n* User Agent and assembly versions\r\n\r\n* Merged PR 291: Add premine reward to SC testnet\r\n\r\n* Add comment to POW reward\r\n\r\n* Update template\r\n\r\n* SmartContractsTest\r\n\r\n* Fix logging bug\r\n\r\n* Fix logging bug\r\n\r\n* v0.10.0 release notes\r\n\r\n* Merged PR 293: Merge from FN\r\n\r\n* Added endpoint to show smart contract logs (#1549)\r\n\r\n* added endpoint to show smart contract logs\r\n\r\n* Added nlog file\r\n\r\n* StandardTransactionPolicy modularity\r\n\r\n* Steps towards modularity\r\n\r\n* WalletController hack removed\r\n\r\n* Dashboard Controller\r\n\r\n* Possible fix\r\n\r\n* Fix for integration test\r\n\r\n* Merged PR 296: Remove SC-specifics from Script\r\n\r\n* Merged PR 298: ScriptTemplateFactory\r\n\r\n* Merged PR 300: Merge from FN\r\n\r\n* Fix bug with SmartContracts CoreNode\r\n\r\n* Merged PR 299: Script Address Reader\r\n\r\n* Factor out an IScriptAddressReader interface\r\n* Define an implementation for the FN and an implementation for Smart Contracts\r\n* Inject into WalletManager\r\n\r\n* Merged PR 301: Remove Smart Contract networks from NBitcoin\r\n\r\n* Merged PR 302: Remove SmartContractConsensusFactory and SmartContractBlockHeader from NBitcoin (2nd)\r\n\r\n* Merged PR 304: Merge from FN\r\n\r\n* Merged PR 303: Remove IsSmartContracts from NBitcoin (3rd)\r\n\r\n* Merged PR 305: Explicit Smart Contract rule registration\r\n\r\nThis is to the remove the LoadcoinviewRule hack. We node explicitly register rules for smart contracts. No more inheriting from POW consensus rules.\r\n\r\n* Merged PR 306: Change visibility of Auction setters\r\n\r\nRelated work items: #1980\r\n\r\n* Merged PR 308: Merge remote-tracking branch 'upstream/master'\r\n\r\nMerge remote-tracking branch 'upstream/master'\r\n\r\n* Merged PR 307: Separate calldata from carrier\r\n\r\n* Reorder serialization from least likely to change > most likely to change\r\n* Separate call data from transaction data in carrier\r\n\r\n* Merged PR 311: Remove SmartContractExecutor base class\r\n\r\nRemoves the executor abstract base class. Duplicates some functionality in implementations. The reason for this is to enable a larger refactoring where eventually the duplicate functionality will be separated into smaller pieces and reused.\r\n\r\n* Fix build\r\n\r\n* Merged PR 312: DI ReflectionVirtualMachine\r\n\r\n* Merged PR 315: Merge from full node master 2018 07 04\r\n\r\nAll unit and integration tests pass. I couldn't test Auction contract deployment as it will cause a fork. I was only able to sync up until block 28848.\r\n\r\n* Merged PR 316: Merge remote-tracking branch 'upstream/master'\r\n\r\nIntegration tests pass. Some unit tests re base cost fail. Will investigate.\r\n\r\n* FN merge review changes\r\n\r\n* Merged PR 313: Separate VmExecutionResult from SmartContractExecutionResult\r\n\r\nSeparates a VmExecutionResult from SmartContractExecutionResult. Part of a larger refactoring to ensure our Types only return things they know about, and nothing more.\r\n\r\n* Merged PR 314: Return results from post-executors rather than assigning directly\r\n\r\n* Merged PR 318: Fix for basefee not being spent\r\n\r\nFix for basefee not being spent\r\n\r\n* Merged PR 319: Fix issue where gas was being spent twice in internal tx executor\r\n\r\nFix issue where gas was being spent twice in internal tx executor\r\n\r\nRelated work items: #2032\r\n\r\n* Merged PR 321: Fix internaltransfers\r\n\r\nFix internaltransfers\r\n\r\n* Merged PR 322: Merge remote-tracking branch 'upstream/master'\r\n\r\nMerge remote-tracking branch 'upstream/master'\r\n\r\n* Merged PR 323: Merge remote-tracking branch 'upstream/master'\r\n\r\nMerge remote-tracking branch 'upstream/master'\r\n\r\n* Revert read.me\r\n\r\n* Merged PR 324: Merge remote-tracking branch 'upstream/master'\r\n\r\nMerge remote-tracking branch 'upstream/master'\r\n\r\n* Full Node review fixes\r\n\r\n* Fix integration test / merge from FN\r\n\r\n* Revert changes to FullNodeBuilder\r\n\r\n* Remove template building from solution\r\n\r\n* Remove template project build from release build\r\n\r\n* Remove unused reference","distinct":false,"url":"https://api.github.com/repos/fassadlr/StratisBitcoinFullNode/commits/bf7c94d0d9959a6be79910ff072d7e805f9fb01e"},{"sha":"1b9752040b98d90ffe5d39cb29a35bad149437fb","author":{"name":"Francois de la Rouviere","email":"6f4bbf3c79085089f01a384adecf2e945e3a4f92@gmail.com"},"message":"Merge remote-tracking branch 'upstream/master' into remove-generatewithstratis","distinct":true,"url":"https://api.github.com/repos/fassadlr/StratisBitcoinFullNode/commits/1b9752040b98d90ffe5d39cb29a35bad149437fb"}]}
{"actor":{"display_login":"fassadlr"}}