center-randomize icon indicating copy to clipboard operation
center-randomize copied to clipboard

Add rye as a package manager.

Open manisha841 opened this issue 10 months ago • 6 comments

  • Added rye config

Use the following commands to set up rye in your system.

  • Install rye in your machine: https://rye-up.com/guide/installation/
  • rye sync
  • rye add package
  • rye run pre-commit install
  • rye run dev

manisha841 avatar Apr 22 '24 14:04 manisha841

I see a lot of effort being put into this, greatly appreciate it. I had not heard about rye before this, not much into python ecosystem, so a few questions -

  • How wide is the adoption of rye as a package manager?
  • skimming through the documentation, rye does appear to be better than pip, in par with modern package managers. But I believe it does not ship with python installation, so for a newbie like me, it would be one more thing to install and learn. For a project of this size would it add value to make this switch?

sapradhan avatar May 05 '24 17:05 sapradhan

I see a lot of effort being put into this, greatly appreciate it. I had not heard about rye before this, not much into python ecosystem, so a few questions -

  • How wide is the adoption of rye as a package manager?
  • skimming through the documentation, rye does appear to be better than pip, in par with modern package managers. But I believe it does not ship with python installation, so for a newbie like me, it would be one more thing to install and learn. For a project of this size would it add value to make this switch?

Rye manages the package, including the virtual environments and Python one needs, so I think it would be much easier to use Rye as a one-stop solution, as one won't need to worry about Python's virtual environments madness.

For a project of this size would it add value to make this switch?

I agree that for the size of this project, it might be a bit cumbersome, but if we think from the dev's perspective, it might be better off with this one with the main rationale being separation and management of the packages, depending on the dev, test, and ... environment and managing the virtualenvs. After all, we should always opt for the betterment direction. Since we are already using the Streamlint app, I am sure there will be many more packages coming in the future. It's always good to have one less tech debt from the get-go.

horrormyth avatar May 05 '24 18:05 horrormyth

@horrormyth @sumanashrestha my install dependencies check has been failing since I no longer have requirements.txt file, can you update this code coverage check please?

manisha841 avatar May 27 '24 14:05 manisha841

@manisha841, if you could add me to your fork, I could then push changes in a separate branch on your fork or maybe to this branch.

horrormyth avatar May 31 '24 20:05 horrormyth

@manisha841 CI should be fixed now; it seems that the coverage is under 80% feel free to make changes

horrormyth avatar Jun 03 '24 19:06 horrormyth

@manisha841 CI should be fixed now; it seems that the coverage is under 80% feel free to make changes

Thank you, I will look into it further.

manisha841 avatar Jun 10 '24 14:06 manisha841