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

pocsuite3 1.9.1 #97076

Closed
wants to merge 2 commits into from
Closed

Conversation

13ph03nix
Copy link
Contributor

action-homebrew-bump-formula


Created with brew bump-formula-pr.

@BrewTestBot BrewTestBot added bump-formula-pr PR was created using `brew bump-formula-pr` python Python use is a significant feature of the PR or issue rust Rust use is a significant feature of the PR or issue labels Mar 17, 2022
@bayandin
Copy link
Member

depends_on "rust" => :build # for cryptography

The formula doesn't have a cryptography resource. Does it still require rust as a dependency to be built or we can drop it / update the comment for it?

@BrewTestBot BrewTestBot removed the rust Rust use is a significant feature of the PR or issue label Mar 17, 2022
@13ph03nix
Copy link
Contributor Author

Thanks! The reason for this problem is that extras_require is not considered when bump-formula-pr automatically updates pip package dependencies.

@13ph03nix
Copy link
Contributor Author

Since extra_require is not necessary, it is just to make the program more perfect. In order to facilitate automatic updates, I removed its related dependencies.

@BrewTestBot
Copy link
Member

🤖 A scheduled task has triggered a merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump-formula-pr PR was created using `brew bump-formula-pr` python Python use is a significant feature of the PR or issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants