added the required modifications

added those changes

added those changes

@SumantxD What I meant was for each of those to be individual modules, so the directory structure would look more like this:

• msg_fields
  • common
    • attr_value.rs
    • mod.rs
  • protocols
  • mod.rs

Could you please adjust the files to match that structure?

@bobozaur I have fixed the directory structure

@bobozaur I have fixed the directory structure

Nice. Good job!

I triggered the CI run, let's wait on that and if it's all good I'll loop in other maintainers for a quick review and then we can merge this.

@bobozaur a lot of checks were unsuccessful in the CI run, will it create problems? if so how can i fix it?

@bobozaur a lot of checks were unsuccessful in the CI run, will it create problems? if so how can i fix it?

It will, yes. The checks are there to ensure the code works as expected (building, testing, etc.).

You can explore the CI jobs and check the errors, as well as testing locally (the best way to figure out what the CI runs is to look in the main.yaml file and maybe search for things such as cargo test).

After you fix the tests et al we can trigger another CI run and see how it behaves.

Closing per inactivity. Feel free to reopen if you would like to continue in this work.