Update outdated discord links - #2581
Closed
taylorferran wants to merge 19 commits into
Closed
Conversation
Signed-off-by: tzchenxixi <tzchenxixi@icloud.com>
…2466) * Add validation to threshold signature split * New commit to re-open PR * Improve error and fix lint --------- Co-authored-by: rehs0y <lyosher@gmail.com>
…s#2476) * duties: clarify & fix typos & fix sync-committee subscription * lower noActionTimeout as it might cause trouble for some tests * fix flaky test & simplify * log as warn: late duty execution * adjustments * deduplicate & simplify * further similifications & clarifications * deadline on parent-context shouldn't be mandatory * set correct duty-handler deadlines * use context.WithoutCancel instead of context.Background * fix typo
* Exporter: clear beacon adapter cache periodically * Refactor functions names * Exporter: refactor to use defer statements
* Fix partial sig messages classification * Exporter: add timestamps to pending partial messages * Exporter: Add tests * Exporter: remove diff noise * Exporter: fix linter * CI: Fix CI to run LFS tests too
* message/validation: log signers on invalid message * add a nil check for SignedSSVMessage
* operator-id: clarify & simplify * fix comment * simplify code a bit --------- Co-authored-by: rehs0y <lyosher@gmail.com>
Signed-off-by: russcoss <russcoss@outlook.com>
* blockchain/beacon/blind.go: utility to ensure and convert full blocks to blinded blocks * ssv/runner/proposer.go: convert blocks to blinded before using as start value * formatting * leaders cache blocks for later submission * todo about removing submission, debug log about full block exposing. * 2562: iurii suggestions * add TODO --------- Co-authored-by: rehs0y <lyosher@gmail.com>
* majority fork protection * fix linter * fix log * fix returned value * fix log text * fix checkConsensusDataSameToOwn logic for aggregator * only check source/target attestation epoch * value checker interface * create a type for SlashingProtectionData * rename ValueCheckF * add a unit test * update comment about spec decision * spec alignment * code review * differ * differ * differ * fix build issue * differ * fix issues after merging * check target root (ssvlabs#2545) * 2358: iurii refactor (ssvlabs#2551) * 2358: iurii refactor * clarify what valCheck is * fix linter * fix spec tests * cleanup * fix marshaling spectest * cleanup ValueChecker marshaling leftovers --------- Co-authored-by: Nikita Kryuchkov <nkryuchkov10@gmail.com> * code review comments * differ * update checks as per SIP --------- Co-authored-by: iurii-ssv <183610124+iurii-ssv@users.noreply.github.com> Co-authored-by: rehs0y <lyosher@gmail.com>
* runners: robust message replay * improve error-message * fix go-routine leaks * fix potential bug with closure * clean-up ctx usage for Committee struct * minor improvements * use logger from log package in tests * address review comments * fix typo * add missing comment * make the following error non-retryable: no decided value * adjust comment and error-message * improve logging to differentiate between debug/error cases * use test/bench logger instead of no-op logger * refactor retryable-error checks * move common queue-handling stuff in a separate file to deduplicate * simplify retryable error-handling * make RetryableError to actually work as intended * improve error-messages * minor cleanup + clarifications * fix the logger.With in a for-loop * simplify isDecidedMsg func and decided check * improve logging * make estimated_time_into_slot dynamic * fix EstimatedTimeIntoSlot * get rid of EstimatedTimeIntoSlot (as it cannot function in logger.With) * adjustments * handle post-consensus errors when there is no decided value in more precise manner (retrying some of those) * testing * improve retryable behavior for ValidatePostConsensusMsg * make code formatting consistent * improve error message * minor clarifications * improve pre/post consensus message validation * clarify error-messages * clarify comment * clarify comment * align with spec * improve error-messages * spec-tests: implement error-codes (resolves broken spec-tests) * fix compilation after rebase * update/fix ssvsigner go.mod deps * use the latest ssv-spec version * log if queue consumer is done due to context done * comittee-runner: fix ctx usage with respect to cancel/deadlines * tighten runnerExpirySlots to make more sense & match scheduler deadline * set runnerExpirySlots = 34 for compatibility with spec-tests * address review comments * go mod tidy * tether retry params to slot-duration * enable errorf linter * misc adjustments * fix tests --------- Co-authored-by: rehs0y <lyosher@gmail.com>
* update spec to latest version and align * ssvsigner: update spec dependency * update ssv node to latest ssvsigner commit --------- Co-authored-by: Yosher Lutzki <lyosher@gmail.com>
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Greptile Overview
Greptile Summary
Updated all Discord invite links in the README from the outdated discord.gg/ssvnetworkofficial to the current discord.gg/5vT22pRBrf.
- Changed Discord badge link (line 12)
- Updated contribution section Discord reference (line 59)
- Modified community section Discord link (line 86)
All three occurrences are now consistent and point to the updated invite link.
Confidence Score: 5/5
- This PR is completely safe to merge with zero risk
- This is a straightforward documentation update that only changes Discord invite URLs in the README. All three occurrences were updated consistently, no code logic is affected, and the changes are purely cosmetic with no potential for runtime errors or functional issues.
- No files require special attention
Important Files Changed
File Analysis
| Filename | Score | Overview |
|---|---|---|
| README.md | 5/5 | Updated Discord invite links from ssvnetworkofficial to 5vT22pRBrf across all three occurrences |
Sequence Diagram
sequenceDiagram
participant Author as PR Author
participant README as README.md
participant Discord as Discord Server
participant Users as End Users
Author->>README: Update Discord invite link
Note over README: Line 12: Badge link<br/>Line 59: Contribution section<br/>Line 86: Community section
README->>README: Replace ssvnetworkofficial<br/>with 5vT22pRBrf
Users->>README: Read documentation
Users->>Discord: Click updated invite link
Discord->>Users: Join SSV community
1 file reviewed, no comments
kchojn
previously approved these changes
Nov 7, 2025
iurii-ssv
previously approved these changes
Nov 7, 2025
zktaiga
previously approved these changes
Nov 7, 2025
Tom-ssvlabs
previously approved these changes
Nov 10, 2025
y0sher
dismissed stale reviews from Tom-ssvlabs, zktaiga, iurii-ssv, and kchojn
November 10, 2025 11:11
The base branch was changed.
Tom-ssvlabs
requested review from
Tom-ssvlabs,
iurii-ssv,
kchojn and
zktaiga
November 10, 2025 11:14
y0sher
previously approved these changes
Nov 10, 2025
kchojn
previously approved these changes
Nov 10, 2025
y0sher
dismissed stale reviews from kchojn and themself
November 10, 2025 11:18
The base branch was changed.
y0sher
previously approved these changes
Nov 10, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
12 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.