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

[README] Clarify note about unneeded dependencies of Poetry #667

Merged

Conversation

homeworkprod
Copy link
Contributor

poetry-core is a dependency of Poetry, of course.

This change should avoid confusion arising from the former wording.

@radoering
Copy link
Member

LGTM. However, it seems you did not check "Allow edits from maintainers". Thus, I cannot rebase your branch and finally cannot merge it. Please rebase your branch yourself or set the checkbox.

Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@homeworkprod
Copy link
Contributor Author

Rebased.

However, it seems you did not check "Allow edits from maintainers".

Sorry about that. I've heard this remark before in another project, but in both cases that checkbox was/is nowhere to be seen; at least not on the PR page itself. Not sure if this is somehow dependent on how actually a PR is created (e.g. directly on a repository vs. off of another repository).

@homeworkprod
Copy link
Contributor Author

According to https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork the reason this doesn't work here is that my PR comes from a non-user-owned fork (I put forks in a separate org so they don't clutter up my "actual" projects, a practice I've copied from somewhere).

@homeworkprod
Copy link
Contributor Author

Also see here for an unresolved discussion: https://github.com/orgs/community/discussions/5634

@radoering radoering merged commit 56c0376 into python-poetry:main Dec 10, 2023
20 checks passed
@homeworkprod homeworkprod deleted the readme-note-on-deps branch December 10, 2023 15:55
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