CedArctic / Chimera

System Remote Control Discord Bot written in Python discord.py

MIT License

87 stars 23 forks source link

Add Linux Support #7

Closed TGlide closed 5 years ago

TGlide commented 5 years ago

1

Instead of creating a separate linux file, I addedd support on chimera_windows.py . I suggest renaming it to chimera maybe?

Some commands require sudo to run on linux, so I added that information on README.md

Some methods I wasn't able to find a replacement to linux. I think TTS would be able by means of a python module,I can look into it too!

I created a helpers.py so I wouldn't make the main code dirty. It only has a helper function that returns the platform the script is running on.

CedArctic commented 5 years ago

Looks interesting. I would love to look into it a bit more, but for now I'm closing this PR as it contains your personal token and channel ID.

CedArctic commented 5 years ago

Edit: Just push a new commit on top to restore the channel ID and token to the defaults and I'll check it out.

TGlide commented 5 years ago

Yes, I saw that, my bad. I think it's all good now!

CedArctic commented 5 years ago

@TGlide Nice work! At the moment I am looking into #8 and thinking of merging that first. Could you then maybe look into integrating what has been introduced with #8 as well?

TGlide commented 5 years ago

Yes, I'll look into it!

TGlide commented 5 years ago

Fixed some conflicts, will take a look at media control

CedArctic commented 5 years ago

@TGlide that's great! In the mean time since this PR has no conflicts, I will merge it and you can create a new one for porting media control and other features. Thanks for contributing!

TGlide commented 5 years ago

Sure thing! Glad I could help :)