Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Remove legacy 'hooks.yaml' file support#553

Merged
asottile merged 1 commit into
masterfrom
remove_legacy_hooks_yaml_support
Jul 26, 2017
Merged

Remove legacy 'hooks.yaml' file support#553
asottile merged 1 commit into
masterfrom
remove_legacy_hooks_yaml_support

Conversation

@asottile

@asottile asottile commented Jul 3, 2017

Copy link
Copy Markdown
Member

I'll wait to merge this until July 24th (6 months since deprecation in 0.12.0).

@asottile asottile requested a review from chriskuehl July 3, 2017 19:12

@chriskuehl chriskuehl left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

\o/

@chriskuehl

chriskuehl commented Jul 4, 2017

Copy link
Copy Markdown
Member

After merging, it might be a good idea to (temporarily, until they fix it) stop listing repos which don't have .pre-commit-hooks.yaml on the website to avoid confusion.

@asottile

asottile commented Jul 4, 2017

Copy link
Copy Markdown
Member Author

Yeah I plan to change the script to not fall back (and comment them out temporarily)

@asottile asottile force-pushed the remove_legacy_hooks_yaml_support branch from 06c1a1d to 7aad84f Compare July 9, 2017 17:20
@asottile asottile force-pushed the remove_legacy_hooks_yaml_support branch from 7aad84f to 86d9ca0 Compare July 13, 2017 17:29
@asottile asottile merged commit 8f86f8f into master Jul 26, 2017
@asottile asottile deleted the remove_legacy_hooks_yaml_support branch July 26, 2017 12:50
droctothorpe pushed a commit to droctothorpe/pre-commit that referenced this pull request Mar 23, 2022
…e-config

[pre-commit.ci] pre-commit autoupdate
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.

2 participants