[CLOSED] Changed CodeInspector.inspectFile to accept provider.scanFile functions that return the result via a promise

[core-ai-bot]

Issue by jdiehl
Tuesday Jan 21, 2014 at 22:27 GMT
Originally opened as adobe/brackets#6606

---

This change enables third party extensions to use the CodeInspector APIs to report errors from external compilers that are triggered via node and thus must be run asynchronously. The change retains backwards compatibility.

---

jdiehl included the following code: https://github.com/adobe/brackets/pull/6606/commits

[core-ai-bot]

Comment by jdiehl
Tuesday Jan 21, 2014 at 22:30 GMT

---

This change would in particular allow the less-autocompile extension to perform proper error reporting.

[core-ai-bot]

Comment by busykai
Tuesday Jan 21, 2014 at 22:57 GMT

---

Good to see that there are many needing this. #5137 and some other were filed against the same problem. Currently there's a PR #6530 pending to accomplish the same (uses slightly different approach).

[core-ai-bot]

Comment by jdiehl
Wednesday Jan 22, 2014 at 17:23 GMT

---

Sorry for not checking the issues for a similar item. I will leave the pull request open for reference. Feel free to close it when it becomes redundant.

[core-ai-bot]

Comment by ingorichter
Friday Mar 14, 2014 at 16:58 GMT

---

@jdiehl Thanks for your contribution. I'm going to close this PR in favor of #6530 since the PR provides the same functionality and we have unit tests for the new behavior.