-
Notifications
You must be signed in to change notification settings - Fork 56
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
Dockerizing environment #2
Comments
Thanks I will look into it |
@entrpn I was just going to open an issue for this :). How do I run the patch file? |
I've create a PR from etrpn's branch to yours here: |
Thanks @GrantHoffman, I realized that in Also FYI - I'm modifying my fork to focus on penny stocks so the patch would be better than pull request from my fork. |
@granthoff1107 you should clone this repo. download the zip file, unzip it and then run
I should have named the file better but that should work. The readme has steps to run it but basically:
Your .env file should have the following:
hope that helps. |
Thanks, this will help deployment alot
…On Tue, Feb 16, 2021, 3:29 AM Juan Acevedo ***@***.***> wrote:
Thanks @GrantHoffman <https://github.com/GrantHoffman>, I realized that
in get_stream_posts I added my username and password for the throwaway
account that I'm using to run this program. I since changed my password lol.
Also FYI - I'm modifying my fork to focus on penny stocks so the patch
would be better than pull request from my fork.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#2 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADFQZHJ73GC5YKYOI6BYP5TS7IUFZANCNFSM4XTL4RCQ>
.
|
@entrpn I have tried to apply the patch as per your instructions, but I got:
|
@alexferrari88 I think those are warnings. If not take a look at this https://stackoverflow.com/questions/14509950/my-diff-contains-trailing-whitespace-how-to-get-rid-of-it |
Surely the first one, I meant the last error
…On Mon, Mar 1, 2021 at 5:23 PM Juan Acevedo ***@***.***> wrote:
@alexferrari88 <https://github.com/alexferrari88> I think those are
warnings. If not take a look at this
https://stackoverflow.com/questions/14509950/my-diff-contains-trailing-whitespace-how-to-get-rid-of-it
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#2 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ALWB5WXLYYUNDSRX72BGM73TBO5QZANCNFSM4XTL4RCQ>
.
|
try You'll get some stuff that doesn't get merged. You'll need to manually merge those changes. |
I agree with this ticket, should be a bit easier to get running |
Hello GonVas,
Nice project. I wrote a Dockerized environment if you are interested in merging it or creating a branch out of it.
I have attached the patch file.
patchfile.zip
The text was updated successfully, but these errors were encountered: