Skip to content
Snippets Groups Projects

Resolve "Implement Web of Trust in a more "rusty" way"

Merged nanocryk requested to merge 28-implement-web-of-trust-in-a-more-rusty-way into dev
All threads resolved!

Closes #28 (closed)

Edited by nanocryk

Merge request reports

Checking pipeline status.

Merged by nanocryknanocryk 7 years ago (Mar 4, 2018 10:31am UTC)

Merge details

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Éloïs
  • Éloïs
  • Éloïs
  • Éloïs
  • Éloïs
  • Éloïs
  • Félicitations, globalement c'est très bien codé et super lisible, on comprend tout de suite ce que fait le code, c'est très agréable a lire, je vais essayer d'en prendre de la graine :p

    Après avoir traité mes remarques pense a prendre en compte les diff proposés par fmt dans tes macros lazy_static! : https://git.duniter.org/nodes/rust/duniter-rs/-/jobs/3372

    Edited by Éloïs
  • Author Contributor

    @librelois Merci.

    Sinon ce sont des diffs dans une autre crate, on ferra une issue pour ça ou on le corrigera dans une issue en rapport avec la crate protocol.

  • nanocryk added 1 commit

    added 1 commit

    • 72463949 - [fix] improvements following MR review

    Compare with previous version

  • nanocryk added 1 commit

    added 1 commit

    • cc3597b4 - [fix] improvements following MR review

    Compare with previous version

  • Éloïs
  • Éloïs
  • nanocryk added 1 commit

    added 1 commit

    • 77592f49 - [enh] optimizations + clippy

    Compare with previous version

  • Éloïs resolved all discussions

    resolved all discussions

  • review OK, tu peut merger :)

  • nanocryk mentioned in commit 8d1ca53a

    mentioned in commit 8d1ca53a

  • merged

  • Please register or sign in to reply
    Loading