FIX: shift to pydantic

[eadaradhiraj]

Modified classes to basemodel of pydantic where possible

[yeger00]

I don't see any changes in the caller functions (i.e. py_lsp_client). Are there any changes needed?
I also added some tests that might fail now, can you please check that as well?

Thanks for you support! Much appreciated

[eadaradhiraj]

I don't see any changes in the caller functions (i.e. py_lsp_client). Are there any changes needed? I also added some tests that might fail now, can you please check that as well?

Thanks for you support! Much appreciated

Of course. Will look into it.

[eadaradhiraj]

I don't see any changes in the caller functions (i.e. py_lsp_client). Are there any changes needed? I also added some tests that might fail now, can you please check that as well?
Thanks for you support! Much appreciated

Of course. Will look into it.

I looked into. But I could not get this converted to pydantic. I will have to do more research on this. The trouble is the the type identifier is pointing to a class that inherits a threading module and keeps throwing an "Unable to generate pydantic-core schema" error. There is perhaps another way to convert it.

[yeger00]

I added an example PR of shifting one class into pydantic. You can take a look and let me know what you thinkg

#40

[yeger00]

I added another class to Pydantic, you can see it here: #42
Let me know if you want to join me in moving the rest of these.

[eadaradhiraj]

I added another class to Pydantic, you can see it here: #42 Let me know if you want to join me in moving the rest of these.

Kindly give me some time.

[yeger00]

No problem, thanks for your effort

[eadaradhiraj]

The test cases are failing. Is this due to changes done in the class. I can look into them

[eadaradhiraj]

assert set(symbol.name for symbol in symbols) == set(expected_symbols) is failing.

symbols is coming up empty.

Should I configure anything?

[eadaradhiraj]

I have changed all I can to the best of my ability. Please check