search

maidsafe / sn_routing

Routing - specialised storage DHT
Other
278 stars 81 forks source link

refactor(core): split up mega-file of 2500 loc #2486

Closed oetyng closed 3 years ago

semgrep-app[bot] commented 3 years ago

Semgrep was unable to comment directly on `{path}

In src/routing/core/messaging/handling/mod.rs:631 Semgrep found a match for "FIXME" (FIXME 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[bot] commented 3 years ago

Semgrep was unable to comment directly on `{path}

In src/routing/core/public_api.rs:256 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[bot] commented 3 years ago

Semgrep was unable to comment directly on `{path}

In src/routing/core/messaging/sending.rs:363 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[bot] commented 3 years ago

Semgrep was unable to comment directly on `{path}

In src/routing/core/messaging/handling/agreement.rs:97 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[bot] commented 3 years ago

Semgrep was unable to comment directly on `{path}

In src/routing/core/messaging/handling/bad_msgs.rs:126 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[bot] commented 3 years ago

Semgrep was unable to comment directly on `{path}

In src/routing/core/messaging/handling/bad_msgs.rs:108 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[bot] commented 3 years ago

Semgrep was unable to comment directly on `{path}

In src/routing/core/messaging/mod.rs:89 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[bot] commented 3 years ago

Semgrep was unable to comment directly on `{path}

In src/routing/core/messaging/sending.rs:173 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