Skip to content

V0.2.1#62

Merged
FL03 merged 10 commits intomainfrom
v0.2.1
Apr 28, 2024
Merged

V0.2.1#62
FL03 merged 10 commits intomainfrom
v0.2.1

Conversation

@FL03
Copy link
Copy Markdown
Member

@FL03 FL03 commented Apr 10, 2024

No description provided.

FL03 added 3 commits March 22, 2024 08:13
Signed-off-by: Joe McCain III <jo3mccain@icloud.com>
Signed-off-by: Joe McCain III <jo3mccain@icloud.com>
Signed-off-by: Joe McCain III <jo3mccain@icloud.com>
@FL03 FL03 added bug Something isn't working enhancement New feature or request rust Improvements or additions that update the Rust code versioning labels Apr 10, 2024
@FL03 FL03 added this to the v0.2.1 milestone Apr 10, 2024
@FL03 FL03 self-assigned this Apr 10, 2024
Comment on lines +62 to +65
Some(ref value) => match fs {
Some(f) => Some(f(value)),
None => None,
},

Check warning

Code scanning / clippy

manual implementation of `Option::map`

manual implementation of `Option::map`
let v = (f)(&self[i]);
result.push(v)
}
return result;

Check warning

Code scanning / clippy

unneeded `return` statement

unneeded `return` statement
Comment thread derive/src/display/mod.rs Fixed
Comment thread derive/src/params/mod.rs Fixed
Comment thread derive/src/params/mod.rs Fixed
Signed-off-by: Joe McCain III <jo3mccain@icloud.com>
Comment thread core/src/utils.rs Fixed
Comment thread core/src/utils.rs
let mut n1: (bool, char) = (first.is_lowercase(), first);

for c in text {
let prev_n1 = n1.clone();

Check warning

Code scanning / clippy

using `clone` on type `(bool, char)` which implements the `Copy` trait

using `clone` on type `(bool, char)` which implements the `Copy` trait
fn name(&self) -> String;

fn slug(&self) -> String {
self.name().to_lowercase().replace(" ", "-")

Check warning

Code scanning / clippy

single-character string constant used as pattern

single-character string constant used as pattern
Comment thread derive/src/enums/mod.rs Fixed
Comment thread derive/src/utils.rs Fixed
Signed-off-by: Joe McCain III <jo3mccain@icloud.com>
Comment thread core/src/utils.rs
pub fn file_to_vec(fp: String) -> Result<Vec<String>, io::Error> {
let file_in = File::open(fp)?;
let file_reader = BufReader::new(file_in);
Ok(file_reader.lines().filter_map(Result::ok).collect())

Check warning

Code scanning / clippy

`filter_map()` will run forever if the iterator repeatedly produces an `Err`

`filter_map()` will run forever if the iterator repeatedly produces an `Err`
Comment thread derive/src/params.rs Fixed
Comment thread derive/src/display.rs Fixed
Comment thread derive/src/utils.rs
let mut n1: (bool, char) = (first.is_lowercase(), first);

for c in text {
let prev_n1 = n1.clone();

Check warning

Code scanning / clippy

using `clone` on type `(bool, char)` which implements the `Copy` trait

using `clone` on type `(bool, char)` which implements the `Copy` trait
FL03 added 2 commits April 27, 2024 13:33
Signed-off-by: Joe McCain III <jo3mccain@icloud.com>
Signed-off-by: Joe McCain III <jo3mccain@icloud.com>
Comment thread derive/src/display.rs
Comment on lines +18 to +19
.find(|attr| attr.path().is_ident("display"))
.is_some()

Check warning

Code scanning / clippy

called `is_some()` after searching an `Iterator` with `find`

called `is_some()` after searching an `Iterator` with `find`
FL03 added 3 commits April 28, 2024 09:28
Signed-off-by: Joe McCain III <jo3mccain@icloud.com>
Signed-off-by: Joe McCain III <jo3mccain@icloud.com>
Signed-off-by: Joe McCain III <jo3mccain@icloud.com>
@FL03 FL03 merged commit 1cc7caa into main Apr 28, 2024
@FL03 FL03 deleted the v0.2.1 branch April 28, 2024 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request rust Improvements or additions that update the Rust code

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants