Skip to content

Prevent duplicate settings block on redeploy - #24

Merged
copelco merged 1 commit into
caktus:mainfrom
tim-schilling:prevent-duplicate-settings
Aug 28, 2026
Merged

Prevent duplicate settings block on redeploy#24
copelco merged 1 commit into
caktus:mainfrom
tim-schilling:prevent-duplicate-settings

Conversation

@tim-schilling

Copy link
Copy Markdown
Contributor

This matches the dsd-flyio approach of calling _check_settings from within _validate_platform.

This was generated with the help of Claude based on some friction points of attempting to deploy to PythonAnywhere repeatedly.

@copelco copelco left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm after fixing the merge conflict/pre-commit!

A PythonAnywhere-specific settings block was already found in settings.py.
That block will be replaced with a fresh copy.
 (yes|no) 

@tim-schilling
tim-schilling force-pushed the prevent-duplicate-settings branch from 7439abc to 5ec9afe Compare August 28, 2026 13:06
This matches the dsd-flyio approach of calling _check_settings
from within _validate_platform.
@tim-schilling
tim-schilling force-pushed the prevent-duplicate-settings branch from 5ec9afe to d701c51 Compare August 28, 2026 13:06
@copelco
copelco merged commit 7aad1cc into caktus:main Aug 28, 2026
2 checks passed
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