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

Settings and dependabot fixes #888

Merged
merged 3 commits into from
Aug 5, 2024

Conversation

dchiller
Copy link
Collaborator

@dchiller dchiller commented Aug 1, 2024

This PR addresses two minor issues:

  1. Fixes an issue in how the "DEVELOPMENT" environment variable was read in the django settings.py file. The result of the os.environ.get call is alway a string, so we need to check it against a string in order to get a boolean value.
  2. It has dependabot check for updates monthly rather than weekly. Since we look for patch updates, this was resulting in a lot of dependabot PR's being opened and then closed much more frequently than we actually update our servers.

It also removes an unused version parameter in the docker-compose.yml file.

@dchiller dchiller requested review from lucasmarchd01 and removed request for lucasmarchd01 August 1, 2024 12:56
@dchiller dchiller merged commit 2b3cbc9 into DDMAL:main Aug 5, 2024
2 checks passed
@dchiller dchiller deleted the settings-and-dependabot-fixes branch August 5, 2024 18:35
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.

2 participants