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

Add missing env variables #166

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from
Open

Conversation

k4rt0fl3r
Copy link

Direct PR on master branch as it's a change on the .env.example.
Added the missing env variables

  • OVISBOT_DB_URL
  • OVISBOT_THIRD_PARTY_COGS_INSTALL_DIR

apogiatzis and others added 3 commits December 5, 2020 17:30
* fix(docs): fix installation instruction (cybermouflons#147)

* feat(ctf): add osint challenge category (cybermouflons#146)

* feat(tools): enable CodeQL scanning (cybermouflons#149)

* feat(ctf): add optional difficulty argument on addchallenge (cybermouflons#148)

* feat(ctf): add optional difficulty argument on addchallenge

* fix: isinstance

* Update ctf.py

* fix: exception message and minor docstring change

Co-authored-by: Kyriakos Georgiou <[email protected]>
Bumps [cryptography](https://github.com/pyca/cryptography) from 3.1 to 3.2.
- [Release notes](https://github.com/pyca/cryptography/releases)
- [Changelog](https://github.com/pyca/cryptography/blob/master/CHANGELOG.rst)
- [Commits](pyca/cryptography@3.1...3.2)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [cryptography](https://github.com/pyca/cryptography) from 3.2 to 3.3.2.
- [Release notes](https://github.com/pyca/cryptography/releases)
- [Changelog](https://github.com/pyca/cryptography/blob/master/CHANGELOG.rst)
- [Commits](pyca/cryptography@3.2...3.3.2)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@k4rt0fl3r k4rt0fl3r changed the base branch from master to dev July 2, 2021 16:01
@k4rt0fl3r
Copy link
Author

Hello @apogiatzis and @kgeorgiou ,

As I don't have write privs on the main repo, could you please go on and merge this change?

Thanks

Copy link
Member

@kgeorgiou kgeorgiou left a comment

Choose a reason for hiding this comment

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

@eioake01 can you please rebase your changes so the unrelated commits/changes go away? 🙏 It should be good to go right after 🚀

@k4rt0fl3r k4rt0fl3r force-pushed the master branch 2 times, most recently from 9bdd352 to 7ec7d61 Compare October 7, 2022 02:23
@k4rt0fl3r
Copy link
Author

should be alright now? 😅

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