Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
jsync-swirlds created a review comment on a pull request on hashgraph/hedera-block-node
unnecessary.
jsync-swirlds created a review comment on a pull request on hashgraph/hedera-block-node
This is bad practice. 1. Copying a constant to a class attribute is an anti-pattern 1. Using a remote default constant when an overload exists to apply defaults _internally_ to the library is an ...
jsync-swirlds created a review on a pull request on hashgraph/hedera-block-node
Still reviewing, but these changes are not correct.
jsync-swirlds created a review on a pull request on hashgraph/hedera-block-node
Still reviewing, but these changes are not correct.
jsync-swirlds created a review comment on a pull request on hashgraph/hedera-block-node
Ugh, this is just super ugly with the excessive wrapping. If we can add a temporary variable or two to fix that, it would be good.
jsync-swirlds created a review comment on a pull request on hashgraph/hedera-block-node
Generally, prefer to _not_ static import methods. It costs little to write `FileUtilities.readFileBytesUnsafe` and is _much_ easier to read and understand without having to simultaneously retain a...
jsync-swirlds created a review comment on a pull request on hashgraph/hedera-block-node
This file should actually be retained. These constants are specific to the simulator and should be passed to the methods in common, rather than relying on defaults in that module. We didn't leave...
jsync-swirlds created a review comment on a pull request on hashgraph/hedera-block-node
There was no reason to expand this.
jsync-swirlds created a review comment on a pull request on hashgraph/hedera-block-node
We should not expand these.
jsync-swirlds created a review comment on a pull request on hashgraph/hedera-block-node
This was correct as it stood, there is no reason to reformat the line. The use of `var` was deliberate to avoid an unnecessary line break. The only reformatting we should be doing is via `./gradl...
jsync-swirlds created a review comment on a pull request on hashgraph/hedera-block-node
I do not believe this is a good change, generally you had to use a Path to get the File in the first place, and it's better to pass in the Path rather than converting the Path to a File and back to...
jsync-swirlds created a review comment on a pull request on hashgraph/hedera-block-node
Don't reformat javadoc like this, please.
jsync-swirlds created a review comment on a pull request on hashgraph/hedera-block-node
Same here, do not try to align descriptions. For Javadoc, we should use one space after the parameter _name_, and 4 space indentation(only) on subsequent lines. note, if your IDE is doing this,...
jsync-swirlds created a review comment on a pull request on hashgraph/hedera-block-node
Please, please, please do _not_ start this horrid pattern. There is no reason to use aligned indentation here, and it's ridiculously costly as people have to constantly reformat lines for simple v...
jsync-swirlds created a review comment on a pull request on hashgraph/hedera-block-node
Javadoc should wrap with shorter lines; it's intended to be readable in github which locks down to a narrow viewport for some incomprehensible reason.
jsync-swirlds created a review on a pull request on hashgraph/hedera-block-node
It appears you ran an IDE reformat on these files without disabling a lot of very bad defaults. It's important to keep in mind that most IDEs do not, and often cannot, correctly format the codebas...
jsync-swirlds created a review on a pull request on hashgraph/hedera-block-node
It appears you ran an IDE reformat on these files without disabling a lot of very bad defaults. It's important to keep in mind that most IDEs do not, and often cannot, correctly format the codebas...
jsync-swirlds created a review comment on a pull request on hashgraph/hedera-block-node
None of these extensions is likely to remain in place forever. The whole point here is to have a method that accepts parameters for file extensions and offer a sane default via an override. The v...
jsync-swirlds created a review comment on a pull request on hashgraph/hedera-block-node
Remove in this case; we shouldn't be using this dependency.
AlfredoG87 closed an issue on hashgraph/hedera-block-node
EPIC: Set-up release and deployment pipeline
```[tasklist] ### Tasks - [ ] https://github.com/hashgraph/hedera-block-node/issues/44 - [ ] https://github.com/hashgraph/hedera-block-node/issues/45 - [ ] https://github.com/hashgraph/hedera-b...
AlfredoG87 created a comment on an issue on hashgraph/hedera-block-node
Yes the original recording does not have keys and its only crypto transfer as you noted @Nana-EC, however is my understanding that @jasperpotts has a new recording that simulates this much better.
AlfredoG87 created a comment on a pull request on hashgraph/hedera-block-node
These are the changes to the protobuf that have happened: https://github.com/hashgraph/hedera-protobufs/compare/block-repeated-fix-tss-message...main
Nana-EC created a comment on an issue on hashgraph/hedera-block-node
@AlfredoG87 have we recorded this? I think the initial estimation did not have realistic accounts with multiple keys etc. If we have the numbers let's note them here and confirm with @jasperpotts a...
AlfredoG87 created a comment on a pull request on hashgraph/hedera-block-node
Blocked by: https://github.com/hashgraph/pbj/issues/263
ata-nas created a review comment on a pull request on hashgraph/hedera-block-node
@AlfredoG87 good question. I was thinking about that today when reviewing #300, since that issue should IMO use this logic (see [this](https://github.com/hashgraph/hedera-block-node/pull/300#discu...
