8000 fix: adds pre-commit to dev group and simplifies ci-lint by alexanderankin · Pull Request #440 · testcontainers/testcontainers-python · GitHub
  • [go: up one dir, main page]
    Skip to content

    fix: adds pre-commit to dev group and simplifies ci-lint #440

    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.

    Sign up for GitHub

    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

    Jump to bottom
    Closed
    alexanderankin wants to merge 2 commits into from
    Closed

    fix: adds pre-commit to dev group and simplifies ci-lint #440

    alexanderankin wants to merge 2 commits into from

    Conversation

    @alexanderankin
    Copy link
    Member
    @alexanderankin alexanderankin commented Mar 3, 2024

    No description provided.

    @alexanderankin alexanderankin changed the title adds pre-commit to dev group and simplifies ci-lint [chore] adds pre-commit to dev group and simplifies ci-lint Mar 3, 2024
    @alexanderankin alexanderankin changed the title [chore] adds pre-commit to dev group and simplifies ci-lint chore: adds pre-commit to dev group and simplifies ci-lint Mar 3, 2024
    @alexanderankin alexanderankin changed the title chore: adds pre-commit to dev group and simplifies ci-lint fix: adds pre-commit to dev group and simplifies ci-lint Mar 3, 2024
    @alexanderankin alexanderankin mentioned this pull request Mar 3, 2024
    Merged
    @totallyzen
    Copy link
    Contributor
    totallyzen commented Mar 4, 2024

    made a choice between this PR and #438 - I preferred the other one since it helped bump dependencies forward a bit.

    closing this as duplicate

    @totallyzen totallyzen closed this Mar 4, 2024
    @totallyzen totallyzen deleted the add_pre-commit_to_dev branch March 4, 2024 08:49
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Reviewers

    No reviews

    Assignees

    No one assigned

    Labels

    None yet

    Projects

    None yet

    Milestone

    No milestone

    Development

    Successfully merging this pull request may close these issues.

    3 participants

    @alexanderankin @totallyzen
    0