Shargon
Shargon
Because `Instruction.Size` is `int` and this change is not required because is already checked, I will close it, if you consider is important, we can reopen it.
Related with https://github.com/neo-project/neo-express/issues/486
> A glance of this pr, no idea what its for. Would be great to have some comments in the code, and description in the pr. https://github.com/neo-project/neo/issues/3145
> 1. We need some number to proof this is worth doing at all. > 2. How often people request this via RPC? is not only requested by rpc, also...
I will use a different approach
> What is current status, @shargon? > > At least some basic hash of block notifications is enough. That is what some protocols do. We can continue with it
I think we should add only tasks that have already been agreed upon, `NotaryService`, `ServiceLayer` for example are still under discussion
> @shargon We had to create a new repository inside the organization. It's not added to the repository itself (afaik). > I think you need to fork it inside Neo...
> @shargon Hey why not use githubs? > > ```yaml > # For most projects, this workflow file will not need changing; you simply need > # to commit it...
> what are you expecting it to do? Besides that? Write our code? Provide an automatic review. It could helps, or not, but we can try.