Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update config_gen.py #190

Merged
merged 1 commit into from
Nov 27, 2021
Merged

Conversation

sesseor
Copy link
Contributor

@sesseor sesseor commented Nov 24, 2021

fix to #189 #188 #186

@digitalftl
Copy link

I've tried this update on Ubuntu 20.4 it works.

@MrPowerScripts
Copy link
Owner

I think it would be good to know why the config parser package causes this issue. What is the difference between the config file generated using configparser, or by generating the lines manually?

@sesseor
Copy link
Contributor Author

sesseor commented Nov 25, 2021

I think it would be good to know why the config parser package causes this issue. What is the difference between the config file generated using configparser, or by generating the lines manually?

i think that the [REDDIT_AUTH] is causing the the problem cuz when i removed it manually the bot imported the environment variable with out a problem and i couldn't make the configparser generate the .env file with out the header [REDDIT_AUTH]

@MrPowerScripts
Copy link
Owner

I don't remember why I used configparser. But it doesn't seem like it's used anywhere else in the app. so it's probably safe to remove for now if it gets this working.

@MrPowerScripts MrPowerScripts merged commit ccc86c8 into MrPowerScripts:master Nov 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants