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

(ci) only install black when running static checks #4036

Merged
merged 3 commits into from
Aug 2, 2023

Conversation

ebr
Copy link
Member

@ebr ebr commented Jul 27, 2023

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimization
  • Documentation Update
  • Community Node Submission

Have you discussed this change with the InvokeAI team?

  • Yes
  • No, because:

Have you updated all relevant documentation?

  • Yes
  • No

Description

Do not install unnecessary dependencies when running static checks in CI

@ebr ebr force-pushed the fix/deps-for-static-checks branch 2 times, most recently from f692c80 to ef31572 Compare July 28, 2023 00:40
@ebr ebr changed the title (ci) do not install all dependencies when running static checks (ci) only install black when running static checks Jul 28, 2023
@ebr ebr force-pushed the fix/deps-for-static-checks branch from ef31572 to 8a55acf Compare July 28, 2023 20:39
@ebr ebr enabled auto-merge (squash) July 28, 2023 23:13
Copy link
Collaborator

@lstein lstein left a comment

Choose a reason for hiding this comment

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

Thanks. Looks good!

Copy link
Collaborator

@psychedelicious psychedelicious left a comment

Choose a reason for hiding this comment

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

ty

@ebr ebr merged commit 7021467 into main Aug 2, 2023
9 checks passed
@ebr ebr deleted the fix/deps-for-static-checks branch August 2, 2023 23:46
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.

3 participants