Skip to content

Refactor tool to support external settings#9

Merged
ericof merged 10 commits into
mainfrom
run-all-checks
Aug 8, 2022
Merged

Refactor tool to support external settings#9
ericof merged 10 commits into
mainfrom
run-all-checks

Conversation

@ericof
Copy link
Copy Markdown
Member

@ericof ericof commented Aug 4, 2022

@ericof ericof temporarily deployed to DOCKER_HUB August 4, 2022 18:13 Inactive
@ericof ericof changed the title Run all checks Refactor tool to support external settings Aug 4, 2022
@ericof ericof temporarily deployed to DOCKER_HUB August 4, 2022 18:35 Inactive
@ericof ericof marked this pull request as ready for review August 4, 2022 18:35
Comment thread README.md
Comment thread README.md
Comment thread README.md
Comment thread README.md
Comment thread Dockerfile Outdated
Comment thread Dockerfile
@ericof ericof temporarily deployed to DOCKER_HUB August 5, 2022 15:55 Inactive
@ericof ericof requested a review from wesleybl August 5, 2022 16:01
@ericof ericof temporarily deployed to DOCKER_HUB August 5, 2022 17:43 Inactive
@ericof ericof temporarily deployed to DOCKER_HUB August 5, 2022 18:41 Inactive
@ericof ericof temporarily deployed to DOCKER_HUB August 5, 2022 19:44 Inactive
Copy link
Copy Markdown
Member

@wesleybl wesleybl left a comment

Choose a reason for hiding this comment

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

Missing now formatters examples in the README

@ericof ericof temporarily deployed to DOCKER_HUB August 5, 2022 21:21 Inactive
@ericof
Copy link
Copy Markdown
Member Author

ericof commented Aug 5, 2022

@wesleybl Documentation updated

@ericof ericof requested a review from wesleybl August 5, 2022 21:24
@mauritsvanrees
Copy link
Copy Markdown
Member

Thanks a lot!

I have updated plone.dexterity from version 1 to 2 of code-quality. PR here:
plone/plone.dexterity#171

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