-
Notifications
You must be signed in to change notification settings - Fork 3
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
Disable Logging #28
Merged
Merged
Disable Logging #28
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Poetry setup
Create build using github action
* Add dummy commit to test GH * Change name of precommit file * Add dummy skeleton to test build * Add Dockerfile * Add production github action * Add author to dockerfile
Fix problem with name in the registry
* Add PR template file * Change semantic of type of change * Add documentation as option
* Rename az registries information * Rename pull request template * Rename PR template
* Test PR template * Update version of discordpy dependency * Initial architecture of bot * Change tox and preconfig files * Test build * Add cog automatical registration * Add discord token as secure env var
* Add unit test for intent registration * Add unit testing for startup * Add test for startup cogs * Create unit testing for formatters * Update constants.py Co-Authored-By: Manuel CH <26100917+akotadi@users.noreply.github.com> * Test precommit * Create unit testing for formatters * Add Readme * Fix typo in readme * Ignore rule to avoid unneccesary comments * Delete unnecesary comments * Move fixture to conftest Co-authored-by: Manuel CH <26100917+akotadi@users.noreply.github.com>
* Add initial base for timeline * Add unit testing for hiring formatter * Add cron testing * Add unit testing for cog * Change formatter access * Migrate cron to enum * Add annoucement channel as secret for deployment * Add start stop cron schedule
* Debug state * Change to PR * Add analytics to container * Add cogs again * Remove path * Enable cogs again * Test * Rename Docker root file * Add module parameter to Dockerfile * Add complete path * Uncomment cogs * Comment hiring * Add logg * Test * Add complete path to cogs * Put release again on merge * Fix tox passing failing tests * Fix tox * Change tox * FIx tox * Tox should trigger * Skip update dependency * Delete install * Add pytest * Return pytest * Test should fail * Change test to build * Uncomment test cog * Revert push to prod to merge * Prepare deploy with different envs * Test dev deploy * Test secrets * Add env vars for deploy * Fix if * Split dev and prod * Comment test phase
* Comment beta command * Modify tests
* Add logging decorator * Test log in az dev * Test in developent environment
* Change policy * Disable push in pull request * Fix lintern problems
* Enable Logging (#24) * Add poetry skeleton * Add precommit file config * Add discord as dependency * Add tox configuration files and pass precommit * Change the author of the project * GitHub actions (#2) Create build using github action * Bot skeleton (#4) * Add dummy commit to test GH * Change name of precommit file * Add dummy skeleton to test build * Add Dockerfile * Add production github action * Add author to dockerfile * Fix docker file not getting image in FROM (#5) * Docker file fix v2 (#6) Fix problem with name in the registry * Add PR template file (#7) * Add PR template file * Change semantic of type of change * Add documentation as option * Rename az registries information (#8) * Rename az registries information * Rename pull request template * Rename PR template * React with reaction (#9) * Test PR template * Update version of discordpy dependency * Initial architecture of bot * Change tox and preconfig files * Test build * Add cog automatical registration * Add discord token as secure env var * Test no cog registration (#10) * Fix problem with relative path (#11) * Fix linter config (#14) * Unit test and documentation for blueprint bot (#12) * Add unit test for intent registration * Add unit testing for startup * Add test for startup cogs * Create unit testing for formatters * Update constants.py Co-Authored-By: Manuel CH <26100917+akotadi@users.noreply.github.com> * Test precommit * Create unit testing for formatters * Add Readme * Fix typo in readme * Ignore rule to avoid unneccesary comments * Delete unnecesary comments * Move fixture to conftest Co-authored-by: Manuel CH <26100917+akotadi@users.noreply.github.com> * Add initial base for timeline (#15) * Add initial base for timeline * Add unit testing for hiring formatter * Add cron testing * Add unit testing for cog * Change formatter access * Migrate cron to enum * Add annoucement channel as secret for deployment * Add start stop cron schedule * Missing root folder name was failing deploy to Azure (#16) * Debug state * Change to PR * Add analytics to container * Add cogs again * Remove path * Enable cogs again * Test * Rename Docker root file * Add module parameter to Dockerfile * Add complete path * Uncomment cogs * Comment hiring * Add logg * Test * Add complete path to cogs * Put release again on merge * Fix tox passing failing tests * Fix tox * Change tox * FIx tox * Tox should trigger * Skip update dependency * Delete install * Add pytest * Return pytest * Test should fail * Change test to build * Uncomment test cog * Revert push to prod to merge * Prepare deploy with different envs * Test dev deploy * Test secrets * Add env vars for deploy * Fix if * Split dev and prod * Comment test phase * Disable beta command (#18) * Comment beta command * Modify tests * Add logging decorator (#20) * Add logging decorator * Test log in az dev * Test in developent environment * Change policy (#23) * Change policy * Disable push in pull request * Fix lintern problems Co-authored-by: Manuel CH <26100917+akotadi@users.noreply.github.com> * Comment logger --------- Co-authored-by: Manuel CH <26100917+akotadi@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Disable logging to avoid not triggering CRON
Type of change
How Has This Been Tested?
Tested in Production directly to ensure it was working.
Checklist: