[CLOSED] Fix for issue #1483. Document file and handler function for each command.

[core-ai-bot]

Issue by lkcampbell
Tuesday Sep 10, 2013 at 21:30 GMT
Originally opened as adobe/brackets#5155

---

Fix for issue #1483. Document file and handler function for each command.

---

lkcampbell included the following code: https://github.com/adobe/brackets/pull/5155/commits

[core-ai-bot]

Comment by redmunds
Wednesday Sep 11, 2013 at 20:45 GMT

---

Done with code review. Just 1 minor comment.

[core-ai-bot]

Comment by lkcampbell
Thursday Sep 12, 2013 at 04:19 GMT

---

@redmunds, code review changes pushed.

[core-ai-bot]

Comment by peterflynn
Thursday Sep 12, 2013 at 15:13 GMT

---

This seems like it will be hard to keep in sync with the actual code -- I'm guessing it will drift out of date fairly quickly. I wonder if it's better to only list the module name? That probably changes less often than the function name, and once you have the module name it's trivial to find the command handler within it (that is, telling you the module name is the part with the most bang for the buck).

[core-ai-bot]

Comment by redmunds
Thursday Sep 12, 2013 at 15:18 GMT

---

@peterflynn The discussion on implementating this was done in issue adobe/brackets#1483, so can you comment over there? I copied your comment over there.

[core-ai-bot]

Comment by redmunds
Thursday Sep 12, 2013 at 18:16 GMT

---

Merging.