Skip to content

Feature/GitHub action Python formatting/linting#11

Closed
thoroc wants to merge 0 commit intoeasydiffusion:mainfrom
thoroc:main
Closed

Feature/GitHub action Python formatting/linting#11
thoroc wants to merge 0 commit intoeasydiffusion:mainfrom
thoroc:main

Conversation

@thoroc
Copy link
Contributor

@thoroc thoroc commented Feb 10, 2023

Demonstration of what a github action for code formatting/linting would do. Currently using the following:

  • Black
  • isort
  • autoflake
  • flake8

All rules subject to discussion.

All code changes due to the rules above applied to it.

actual_file_name = file_name(i) if callable(file_name) else f"{file_name}_{i}"
path = os.path.join(dir_path, actual_file_name)

if output_format.lower() == 'embed' and file_format.lower() == 'png':
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you please rebase your branch with the latest changes in main? I think it's missing the recent changes for metadata embedding (and is therefore removing these changes). Thanks

@cmdr2
Copy link
Contributor

cmdr2 commented Feb 13, 2023

Thanks, looks good. Please pull the latest changes from main to your branch (to avoid removing the new code for metadata embedding). Otherwise this is ready to merge.

@thoroc thoroc closed this Feb 13, 2023
@thoroc
Copy link
Contributor Author

thoroc commented Feb 14, 2023

Apologies I had to close this PR. I'll need to reopen it with the rebase from cb49807

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