Refactor traits.rs so that it is only traits

Open Beginner friendly
#148 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
72/100
Issue type
Refactor
Clarity
Mostly clear
Activity status
Active
Tech stack
rust
Domain
backend

Research direction

Start with traits.rs and inspect the related impls for SecurityStrength. Compare the module layout used by key_material and suspendable_state, then move SecurityStrength into an appropriate security_strength module. Done means traits.rs contains only traits and the project still builds and tests pass.

Written by the indexing model from the issue text.

Description

good first issue refactor

At the moment, that means moving

pub enum SecurityStrength

and its related impls to a different file. Maybe the right shape is to make a mod bouncycastle-core::security_strength, the same way we treat key_material and suspendable_state.

Dominant language
Rust
Stars
25
Forks
18
Avg merge
16h 38m
Merged PRs (30d)
3

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from bcgit/bc-rust

All issues in bcgit/bc-rust

Similar issues

More Rust issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.