Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update and upgrade dependencies #224

Merged
merged 1 commit into from
Oct 5, 2023

Conversation

mythrnr
Copy link
Contributor

@mythrnr mythrnr commented Oct 5, 2023

What does this change?

  • Update version of dependencies.
    • Note: syn crate is updated from 1.x to 2.x
  • Remove unused deps using cargo-machete and cargo-udeps.
    • Enable to check deps is unused with make check-deps.
  • Now check the attribute is nested by raiden( ... ) when finding derive attribute (table_name etc ).

References

None.

Screenshots

None.

What can I check for bug fixes?

None.

@@ -8,7 +10,7 @@ impl NextToken {
Self(token.into())
}
pub fn into_attr_values(self) -> Result<super::AttributeValues, super::RaidenError> {
let decoded = match base64::decode(self.0) {
let decoded = match STANDARD.decode(self.0) {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

base64::decode and base64::encode is deprecated.
https://docs.rs/base64/0.21.4/base64/fn.decode.html

Copy link
Member

@bokuweb bokuweb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bokuweb bokuweb merged commit 1c45996 into raiden-rs:master Oct 5, 2023
1 check passed
@mythrnr mythrnr deleted the chore/update-deps branch October 17, 2023 03:28
bokuweb added a commit that referenced this pull request Oct 19, 2023
bokuweb added a commit that referenced this pull request Oct 19, 2023
bokuweb added a commit that referenced this pull request Oct 19, 2023
bokuweb added a commit that referenced this pull request Oct 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants