Skip to content
This repository has been archived by the owner on Jan 15, 2019. It is now read-only.

Task: Clear TODO comments across all files. #171

Closed
9 of 10 tasks
kjellr opened this issue Oct 19, 2018 · 1 comment
Closed
9 of 10 tasks

Task: Clear TODO comments across all files. #171

kjellr opened this issue Oct 19, 2018 · 1 comment

Comments

@kjellr
Copy link
Collaborator

kjellr commented Oct 19, 2018

#167 introduced a number of linting fixes to help us pass the Travis tests, but it also opened up a number of TODOs. These are mostly just file comments that need to be updated, but a few are just a little more complicated than that:

template-parts/post/discussion-meta.php

  • 4 instances


/* translators: TODO comment on placeholders */

/* translators: TODO comment on placeholders */

/* translators: TODO comment on placeholders */

inc/template-tags.php

  • 7 instances

/* translators: TODO comment on placeholders */

/* translators: TODO comment on placeholders */

/* translators: TODO comment on placeholders */

/* translators: TODO comment on placeholders */

/* translators: TODO comment on placeholders */

/* translators: TODO comment on placeholders */

/* translators: TODO comment on placeholders */

classes/class-twentynineteen-walker-comment.php

  • 3 instances



/* translators: TODO comment on placeholders */

inc/ui-icons.php

  • 1 instance — this file is otherwise empty, so may be okay to delete. 🤔

* TODO: File doc comment.

js/customizer.js

  • 1 instance

// TODO: Remove "jshint ignore:line" below once $ is actually used.

template-parts/header/entry-header.php

  • 1 instance

template-parts/post/author-info.php

  • 1 instance

* TODO: File doc comment.

footer.php

  • 1 instance

<?php /* translators: TODO comment on placeholders */ ?>

template-parts/header/site-branding.php

  • 1 instance

classes/class-twentynineteen-svg-icons.php

  • 1 instance

@khleomix
Copy link
Contributor

@kjellr working on this

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants