Skip to content

Implement post-commit - #1415

Merged
asottile merged 1 commit into
pre-commit:masterfrom
ModischFabrications:post_commit
Apr 28, 2020
Merged

asottile merged 1 commit into
pre-commit:masterfrom
ModischFabrications:post_commit

Conversation

@ModischFabrications

@ModischFabrications ModischFabrications commented Apr 26, 2020 •

Copy link
Copy Markdown
Contributor

Implement post-commit as an additional git hook.

This will close #1411.

@ModischFabrications

Copy link
Copy Markdown
Contributor Author

Oh, I also added *.idea to the gitignore to prevent intellij from adding stuff that's irrelevant, feel free to remove it if it's unwanted.

@asottile

Copy link
Copy Markdown
Member

Oh, I also added *.idea to the gitignore to prevent intellij from adding stuff that's irrelevant, feel free to remove it if it's unwanted.

just a general tip for open source, you should never touch .gitignore for other people's projects

Comment thread pre_commit/commands/hook_impl.py
Comment thread .gitignore Outdated
Comment thread pre_commit/commands/hook_impl.py Outdated
Comment thread pre_commit/commands/run.py Outdated
@ModischFabrications
ModischFabrications marked this pull request as ready for review April 26, 2020 20:28
@ModischFabrications

ModischFabrications commented Apr 26, 2020 •

Copy link
Copy Markdown
Contributor Author

Ready for review, should be mergeable.

Comment thread tests/commands/install_uninstall_test.py Outdated
Comment thread tests/commands/install_uninstall_test.py Outdated
Comment thread tests/commands/install_uninstall_test.py Outdated
Comment thread tests/commands/install_uninstall_test.py Outdated
Comment thread tests/commands/install_uninstall_test.py Outdated
Comment thread tests/commands/run_test.py Outdated
Comment thread pre_commit/commands/run.py Outdated
Comment thread tests/commands/install_uninstall_test.py Outdated
Comment thread tests/commands/install_uninstall_test.py Outdated
Comment thread tests/commands/install_uninstall_test.py Outdated
@ModischFabrications

Copy link
Copy Markdown
Contributor Author

alright, any last objections? I think it's done.

Comment thread tests/commands/install_uninstall_test.py
@asottile

Copy link
Copy Markdown
Member

also, please stop saying "it's ready to merge" or "it's done" -- it sounds very pretentious and you don't decide when it's done, the project does

Comment thread tests/commands/install_uninstall_test.py Outdated
@asottile
asottile merged commit ec6ba99 into pre-commit:master Apr 28, 2020
@ModischFabrications
ModischFabrications deleted the post_commit branch April 28, 2020 18:24
@asottile

Copy link
Copy Markdown
Member

thanks again, this has been released as part of v2.4.0 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

allow post-commit hook

2 participants