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

Improve warning for pip-compile if no --allow-unsafe was passed #1867

Merged
merged 2 commits into from May 11, 2023

Conversation

chrysle
Copy link
Contributor

@chrysle chrysle commented May 10, 2023

Based on a suggestion from and fixes #1337

Contributor checklist
  • Provided the tests for the changes.
  • Assure PR title is short, clear, and good to be included in the user-oriented changelog
Maintainer checklist
  • Assure one of these labels is present: backwards incompatible, feature, enhancement, deprecation, bug, dependency, docs or skip-changelog as they determine changelog listing.
  • Assign the PR to an existing or new milestone for the target version (following Semantic Versioning).

@chrysle chrysle added the enhancement Improvements to functionality label May 10, 2023
@webknjaz
Copy link
Member

Look like GitHub is drunk today — there's a lot of internal GHA crushes. It might be time to take a break and come back to restarting the jobs later today or tomorrow..

piptools/writer.py Outdated Show resolved Hide resolved
Co-authored-by: Sviatoslav Sydorenko <wk.cvs.github@sydorenko.org.ua>
@chrysle
Copy link
Contributor Author

chrysle commented May 11, 2023

Looks like GitHub is cured again ;-)

@webknjaz webknjaz enabled auto-merge May 11, 2023 16:55
@webknjaz webknjaz merged commit 244ac03 into jazzband:main May 11, 2023
42 checks passed
@chrysle chrysle deleted the pip-compile-no-allow-unsafe-warning branch May 11, 2023 19:04
@atugushev atugushev added this to the 6.14.0 milestone Jun 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improvements to functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve --allow-unsafe warning comment
3 participants