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

started implementation of vars.py #196

Merged
merged 3 commits into from
Oct 5, 2021
Merged

started implementation of vars.py #196

merged 3 commits into from
Oct 5, 2021

Conversation

eitozx
Copy link
Contributor

@eitozx eitozx commented Oct 4, 2021

Overview

  • Added .env.example to make it easy for contributors to understand .env structure (formality lol).
  • Implementation of vars.py.
  • Small Fixes (maybe?)
  • Updated requirements.txt with some modules that were not listed.

Note: Implementation of vars.py isn't completed yet (and i'm not sure either)

@Dhruvacube Dhruvacube added bug Something isn't working dependencies Pull requests that update a dependency file enhancement New feature or request hacktoberfest-accepted help wanted Extra attention is needed labels Oct 4, 2021
@Dhruvacube Dhruvacube linked an issue Oct 4, 2021 that may be closed by this pull request
@Dhruvacube Dhruvacube self-requested a review October 4, 2021 11:36
Copy link
Member

@Dhruvacube Dhruvacube left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @eitozx thanks for this amazing contribution. I see you have taken time to contribute to my broken v2.0 of Minato Namikaze, I appreciated these contribution.

But see I have made some review comments please check those and change it accordingly :)

minato_namikaze/discordbot.py Show resolved Hide resolved
minato_namikaze/discordbot.py Outdated Show resolved Hide resolved
requirements.txt Outdated Show resolved Hide resolved
requirements.txt Outdated Show resolved Hide resolved
requirements.txt Outdated Show resolved Hide resolved
minato_namikaze/discordbot.py Outdated Show resolved Hide resolved
.env.example Show resolved Hide resolved
@eitozx
Copy link
Contributor Author

eitozx commented Oct 4, 2021

gotcha.

@Dhruvacube
Copy link
Member

Dhruvacube commented Oct 4, 2021

@eitozx just ping me when done

@eitozx
Copy link
Contributor Author

eitozx commented Oct 4, 2021

@Dhruvacube done

Copy link
Member

@Dhruvacube Dhruvacube left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay , Thanks :) Approved. Also thanks for the valuable contribution.

@Dhruvacube Dhruvacube merged commit b05c28c into Creatrix-Net:master Oct 5, 2021
@Dhruvacube Dhruvacube added this to the v2.0 milestone Oct 27, 2021
@Dhruvacube
Copy link
Member

@all-contributors please add @eitozx for Infrastructure

@allcontributors
Copy link
Contributor

@Dhruvacube

I've put up a pull request to add @eitozx! 🎉

Dhruvacube added a commit that referenced this pull request Jun 23, 2022
started implementation of vars.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working dependencies Pull requests that update a dependency file enhancement New feature or request hacktoberfest-accepted help wanted Extra attention is needed
Projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

enums implementation of var.py
2 participants