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

Incorrect Indentation Level for the return True Statement #892

Closed
markuskuusisto opened this issue Nov 1, 2023 · 1 comment · Fixed by #897
Closed

Incorrect Indentation Level for the return True Statement #892

markuskuusisto opened this issue Nov 1, 2023 · 1 comment · Fixed by #897
Assignees

Comments

@markuskuusisto
Copy link

Return true statement is tied to verbose flag which likely is not intended behaviour

if stored_hash and stored_hash.strip() == existing_hash.strip():
    if self.verbose:
        print(f"Skipping {path}, no changes detected.")
        return True

Should be:

if stored_hash and stored_hash.strip() == existing_hash.strip():
    if self.verbose:
        print(f"Skipping {path}, no changes detected.")
    return True

@pamelafox pamelafox mentioned this issue Nov 2, 2023
@pamelafox
Copy link
Collaborator

thank you, sent out a fix!

@pamelafox pamelafox self-assigned this Nov 2, 2023
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 a pull request may close this issue.

2 participants