Skip to content
This repository has been archived by the owner on Jun 25, 2021. It is now read-only.

Remove proofchain from all messages and send only when necessary #2523

Merged
merged 7 commits into from Jun 1, 2021

Conversation

Yoga07
Copy link
Contributor

@Yoga07 Yoga07 commented May 25, 2021

No description provided.

@semgrep-app
Copy link
Contributor

semgrep-app bot commented May 25, 2021

Semgrep was unable to comment directly on `{path}

In src/messages/mod.rs:289
Semgrep found a match for "TODO" (TODO from basic)

This comment is advisory. You do not need to address it before merging this pull request.
How do I suppress this? · Report a bug · Manage policy

@semgrep-app
Copy link
Contributor

semgrep-app bot commented May 25, 2021

Semgrep was unable to comment directly on `{path}

In src/messages/mod.rs:298
Semgrep found a match for "TODO" (TODO from basic)

This comment is advisory. You do not need to address it before merging this pull request.
How do I suppress this? · Report a bug · Manage policy

@semgrep-app
Copy link
Contributor

semgrep-app bot commented May 25, 2021

Semgrep was unable to comment directly on `{path}

In src/messages/mod.rs:310
Semgrep found a match for "TODO" (TODO from basic)

This comment is advisory. You do not need to address it before merging this pull request.
How do I suppress this? · Report a bug · Manage policy

@semgrep-app
Copy link
Contributor

semgrep-app bot commented May 25, 2021

Semgrep was unable to comment directly on `{path}

In src/messages/mod.rs:273
Semgrep found a match for "TODO" (TODO from basic)

This comment is advisory. You do not need to address it before merging this pull request.
How do I suppress this? · Report a bug · Manage policy

src/messages/mod.rs Outdated Show resolved Hide resolved
src/messages/mod.rs Outdated Show resolved Hide resolved
src/messages/mod.rs Outdated Show resolved Hide resolved
src/messages/mod.rs Outdated Show resolved Hide resolved
src/routing/core/anti_entropy.rs Outdated Show resolved Hide resolved
src/routing/core/messaging/handling/bad_msgs.rs Outdated Show resolved Hide resolved
src/routing/core/messaging/sending.rs Show resolved Hide resolved
@Yoga07 Yoga07 force-pushed the remove-proofchain branch 4 times, most recently from 6977855 to a3bec08 Compare May 31, 2021 11:16
src/messages/variant.rs Outdated Show resolved Hide resolved
src/messages/mod.rs Outdated Show resolved Hide resolved
src/messages/mod.rs Outdated Show resolved Hide resolved
src/routing/core/messaging/handling/mod.rs Show resolved Hide resolved
src/routing/core/mod.rs Outdated Show resolved Hide resolved
Cargo.toml Outdated Show resolved Hide resolved
src/messages/mod.rs Show resolved Hide resolved
@Yoga07 Yoga07 merged commit 056766c into maidsafe:master Jun 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants