lfglabs-dev / api.starknet.id

StarknetID API written in Rust.
2 stars 5 forks source link

Fix warnings #103

Open Marchand-Nicolas opened 6 days ago

Marchand-Nicolas commented 6 days ago

Description đź“ą

Fix the warnings when using cargo build, such as “use of deprecated method” warnings.
Several warnings are being raised during the build process when using cargo build. These warnings may include the use of deprecated methods or features. This issue aims to resolve these warnings to ensure a clean build process.

Proposed Actions 🛠️

Here’s a checklist of actions to follow for resolving this issue:

  1. Fork and Create Branch:
    Fork the repository and create a new branch using the issue number:

    git checkout -b fix-[issue-number]
  2. Implement Changes:

    • [ ] Identify all warnings generated by cargo build.
    • [ ] Locate the source of each warning in the codebase.
    • [ ] Update the code to use recommended or non-deprecated alternatives.
    • [ ] If you think some warnings can be ignored, such as the async_warning function that is currently unused, but that we'll probably use in the future, you can leave them.
  3. Run Tests and Commit Changes:
    Make sure your changes don't break existing functionality and commit with a clear message:

    git commit -m "Fix: Resolved warnings during cargo build"

Required đź“‹

To keep our workflow smooth, please make sure you follow these guidelines:

Thank you for your contribution 🙏

⚠️ WARNING: Failure to follow the requirements above may result in being added to the OnlyDust blacklist, affecting your ability to receive future rewards.

anonfedora commented 6 days ago

I'm a fullstack Blockchain developer. Will resolve these warnings to ensure a clean build process by reviewing and upgrading deprecated methods as required.

OWK50GA commented 1 day ago

Is it okay if I tackle this? I'm a Cairo Smart Contract Developer

Emmanex01 commented 1 day ago

Is it okay if I tackle this?

Michaelkingsdev commented 1 day ago

Can I be assigned to this?

Benjtalkshow commented 1 day ago

I am a Full Stack Developer with a strong background in blockchain and extensive experience in Next.js/React, TypeScript, and Rust. I’ve made over 45 contributions to over 15 projects in the OnlyDust ecosystem, solving complex issues and delivering efficient, scalable solutions.

I can handle this task. ETA 48hrs.

josephchimebuka commented 1 day ago

Would love to tackle this!

0xdevcollins commented 1 day ago

Would love to tackle this!

mimisavage commented 1 day ago

Can I attempt this issue?

pheobeayo commented 1 day ago

Can I start working on this? I will create my first PR in a day.

hossamanany commented 1 day ago

I’m interested in this one.

suhas-sensei commented 1 day ago

I’d like to work on this.

ooochoche commented 1 day ago

I've gone through the description and the requirements and can fix warning to meet the proposed actions if assigned the task

pheobeayo commented 1 day ago

Will get right on it