Skip to content

Update documentation link for pdm, use recommended includes/excludes #4671

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

Merged
merged 1 commit into from
Jun 20, 2025

Conversation

akaIDIOT
Copy link
Contributor

Update deeplink to PDM's documentation surrounding including / excluding files. Documentation also slightly contradicts current ignored files, it recommends to include project-local configuration in pdm.toml (not .pdm.toml).

Reasons for making this change

After using this ignore-file, I noticed it mentioned .pdm.toml, while my local pdm created pdm.toml.
Looking up the difference, noticed that the included / excluded files didn't match the recommendations in the project's documentation.

Links to documentation supporting these rule changes

Merge and Approval Steps

  • Confirm that you've read the contribution guidelines and ensured your PR aligns
  • Ensure CI is passing
  • Get a review and Approval from one of the maintainers

@akaIDIOT akaIDIOT requested a review from a team as a code owner June 16, 2025 14:42
Copy link
Collaborator

@wirecat wirecat left a comment

Choose a reason for hiding this comment

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

Thank you for contributing!

@wirecat wirecat merged commit 975d0c3 into github:main Jun 20, 2025
@akaIDIOT akaIDIOT deleted the patch-python-pdm branch June 21, 2025 09:29
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