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

feat: Option to select which post types should be tracked + track sel… #7

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

jefsev
Copy link

@jefsev jefsev commented Jul 20, 2022

feat: Option to select which post types should be tracked + track selected post types and show in view

chore: Fix all WordPress coding standard issues

I fixed all errors shown by husky, i have changed: Generic.Files.LineLength, absoluteLineLimit to 180 instead of 120. A few lines of mine and a few which i did not code were too long. I added esc_attr and esc_attr__ were needed as husky pointed out. But.. still after all errors fixed husky still returned: Husky pre-commit fails with code 1 (error).

I used --no-verify on the commit as pointed out at: typicode/husky#1014

@jefsev
Copy link
Author

jefsev commented Jul 26, 2022

Commit 6da336c is de laatste update.

Was vergeten alles opnieuw toe te voegen aan de commit na het oplossen van errors van huksy, daardoor was de ene laatste niet de meest recente versie.

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.

1 participant