Rasa: consider replacing os.path uses with pathlib

Created on 26 Mar 2019  路  15Comments  路  Source: RasaHQ/rasa

The standard python library pathlib makes interactions with paths much easier. We should consider replacing the old os.path with the pathlib functions.

area low type

Most helpful comment

Hi, @silvasara and I would like to work on this issue. Is this still relevant?

All 15 comments

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Hi, this issue is opened to community?

Hi @icaropires,
Sure! Do you want to take it on? That would be amazing! I would just suggest that we do it in multiple smaller PRs instead of one big one.

@wochinge Great! I'll do it :smiley:

Awesome! Thanks a lot! Just tag me when you have got one PR ready 馃帀

How is it going @icaropires ?

@wochinge Sorry, I've been busy but I'll be working on this on the next week!

@icaropires How is it going?

Hi @wochinge , I'm very sorry for the delay, I've gotten busier than I expected and that's why I couldn't finish this issue yet. And, unfortunately, I will be very busy for at least another month.

As this issue requires a massive amount of changes, I will share how far I did (through a draft PR) and I hope it will be useful for someone.

I expect to be able to contribute again soon. Sorry again for the delay.

Thanks @icaropires the draft is super helpful !

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

This issue has been automatically closed due to inactivity. Please create a new issue if you need more help.

Hi, @silvasara and I would like to work on this issue. Is this still relevant?

Hi @RomuloSouza , it's still relevant 馃憤

Was this page helpful?
0 / 5 - 0 ratings