Skip to content
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.

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

SVN Ignore: add last pre-commit check file to the list #13191

Merged
merged 1 commit into from
Aug 6, 2019

Conversation

jeherve
Copy link
Member

@jeherve jeherve commented Aug 6, 2019

Changes proposed in this Pull Request:

  • We don't need to ship this file to WordPress.org, so we can add it to the list.

Testing instructions:

  • Not much to test here.

Proposed changelog entry for your changes:

  • None

@jeherve jeherve added [Status] Needs Review To request a review from fellow Jetpack developers. Label will be renamed soon. [Type] Janitorial [Pri] Normal labels Aug 6, 2019
@jeherve jeherve added this to the 7.7 milestone Aug 6, 2019
@jeherve jeherve requested review from brbrr and a team August 6, 2019 11:13
@jeherve jeherve self-assigned this Aug 6, 2019
@jetpackbot
Copy link

Thank you for the great PR description!

When this PR is ready for review, please apply the [Status] Needs Review label. If you are an a11n, please have someone from your team review the code if possible. The Jetpack team will also review this PR and merge it to be included in the next Jetpack release.

Scheduled Jetpack release: September 3, 2019.
Scheduled code freeze: August 27, 2019

Generated by 🚫 dangerJS against 8e087b2

@dereksmart dereksmart merged commit a2c2988 into master Aug 6, 2019
@dereksmart dereksmart deleted the add/pre-commit-svnignore branch August 6, 2019 16:55
@matticbot matticbot added [Status] Needs Changelog and removed [Status] Needs Review To request a review from fellow Jetpack developers. Label will be renamed soon. labels Aug 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants