Skip to content

Fixed path of the gen cpp and compile_flags files. #29

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

Merged
merged 2 commits into from
Jul 27, 2020

Conversation

kittaakos
Copy link
Contributor

What it does:

#28 has fixed the location of the ino file but the generated cpp and
compile_flags.txt were not moved into the sketch folder.
Touching the ino file triggered a 'textDocument/didChange' event,
but the updateFileData call failed as the files were not found.

This commit moves the files into the appropriate location.

For reviewers:
It would be great to review this PR #28. Apparently, I fixed the location of the sketch file but not the cpp and the compiler_flags.txt.

Signed-off-by: Akos Kitta [email protected]

#28 has fixed the location of the `ino` file but the generated `cpp` and
`compile_flags.txt` were not moved into the sketch folder.
Touching the `ino` file triggered a `'textDocument/didChange'` event,
but the `updateFileData` call failed as the files were not found.

This commit moves the files into the appropriate location.

Signed-off-by: Akos Kitta <[email protected]>
@kittaakos kittaakos requested a review from spoenemann July 23, 2020 12:32
@kittaakos
Copy link
Contributor Author

I'm going to remove <br> in the PR: #27 (comment)

Theia cannot handle them.

Signed-off-by: Akos Kitta <[email protected]>
@kittaakos
Copy link
Contributor Author

Thank you for the review, @spoenemann 👍

@kittaakos kittaakos merged commit 580d370 into master Jul 27, 2020
@cmaglie cmaglie deleted the fix-cpp-and-compile-flags-files-path branch September 15, 2021 13:11
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.

2 participants