4 ms·

Looks interesting! I'm looking forward to trying it. The annotation syntax looks powerful but do people really edit thier code to adjust priorities and assig

by tln 8y ago

Looks interesting! I'm looking forward to trying it. The annotation syntax looks powerful but do people really edit thier code to adjust priorities and assignees? On a related note, I review `TODO` in PRs and consider whether to allow them in the codebase; and `DEBUG` to indicate code that shouldn't even be checked in. Does anyone else have a codetag for this? DEBUG doesn't seem to be a well-known codetag, and I'd prefer to use something other people use.

aviaviavi 8y ago
> Does anyone else have a codetag for this? DEBUG doesn't seem to be a well-known codetag, and I'd prefer to use something other people use. My team now uses `TODO(#pr)`. The #pr tag meaning it should be taken care of in this pull request, but I don't think that's common. Though maybe if toodles gets some use it will become more common :)
- 0xffff2 8y ago
  Do you open PR's before you start writing code? How do you know what number to fill in?
  - aviaviavi 8y ago
    Ah sorry that was a little confusing. "#" signifies a tag / label in toodles, so #pr means adding the literal "pr" tag to the todo. So we're not adding a PR number.
camtarn 8y ago
We used FIXME to denote temporary changes that shouldn't be checked in, or sections which hadn't been written yet, and had a Git hook which stopped code being committed with that tag. (It also looked for 'console.log' and 'debugger' statements in our JS and blocked those, too.)
- tln 8y ago
  FIXME is definitely a recognizable codetag, I'm glad to hear other people have found a similar use.
adrianmsmith 8y ago
FWIW I have seen tag NOCOMMIT used to indicate that a commit should be rejected by a pre-commit hook. Useful when you're putting in debug logging statements you want to remove later, or hard-coding your own credentials during development with the intent on making them configurable later, etc.
- tln 8y ago
  Exactly! Thank you

by tln 8y ago

aviaviavi 8y ago

> Does anyone else have a codetag for this? DEBUG doesn't seem to be a well-known codetag, and I'd prefer to use something other people use. My team now uses `TODO(#pr)`. The #pr tag meaning it should be taken care of in this pull request, but I don't think that's common. Though maybe if toodles gets some use it will become more common :)

0xffff2 8y ago
Do you open PR's before you start writing code? How do you know what number to fill in?
- aviaviavi 8y ago
  Ah sorry that was a little confusing. "#" signifies a tag / label in toodles, so #pr means adding the literal "pr" tag to the todo. So we're not adding a PR number.

camtarn 8y ago

We used FIXME to denote temporary changes that shouldn't be checked in, or sections which hadn't been written yet, and had a Git hook which stopped code being committed with that tag. (It also looked for 'console.log' and 'debugger' statements in our JS and blocked those, too.)

tln 8y ago
FIXME is definitely a recognizable codetag, I'm glad to hear other people have found a similar use.

adrianmsmith 8y ago

FWIW I have seen tag NOCOMMIT used to indicate that a commit should be rejected by a pre-commit hook. Useful when you're putting in debug logging statements you want to remove later, or hard-coding your own credentials during development with the intent on making them configurable later, etc.

tln 8y ago
Exactly! Thank you