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

remove mention of python.json #526

Merged
merged 1 commit into from
Feb 11, 2025
Merged

remove mention of python.json #526

merged 1 commit into from
Feb 11, 2025

Conversation

Kirbeerus
Copy link
Contributor

@Kirbeerus Kirbeerus commented Feb 11, 2025

Describe your PR

_Summarize your changes here :
The python.json file is a legacy tool we do not recommend to use anymore. Instead, we say to use CC_PYTHON_MODULE.
There was still a mention of using python.json in the Django documentation that is not needed anymore.
There is no need to leave this mention of the file as it would only lead to confusion since we don't document it in the other part of the doc.

Checklist

Reviewers

Who should review these changes? @CleverCloud/reviewers

Copy link

github-actions bot commented Feb 11, 2025

👋` Review app deleted

You closed this PR and deleted the review app.

@Kirbeerus Kirbeerus linked an issue Feb 11, 2025 that may be closed by this pull request
1 task
@Kirbeerus Kirbeerus requested a review from a team February 11, 2025 15:26
Copy link
Collaborator

@davlgd davlgd left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for this!

@davlgd davlgd merged commit 71de56d into main Feb 11, 2025
4 checks passed
@davlgd davlgd deleted the pythonJsonRemove branch February 11, 2025 16:16
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.

Doc: add documentation for python.json
2 participants