Skip to content
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

add Lucee Step Debugger Installation Guide #1447

Merged
merged 4 commits into from
Jul 14, 2024

Conversation

andreasRu
Copy link
Contributor

No description provided.

@zspitzer
Copy link
Member

zspitzer commented Jul 11, 2024

lovely, would you like to link this to the getting started page?

https://docs.lucee.org/guides/getting-started/lucee-editors-IDEs.html

that page could also be re-prioritised, cfeclipse can be removed, dictionary stuff can
go at the bottom, this content can be linked via the VS Code section, which should be first

the path mappings are trickey and I got stuck myself, the example should include a mapping fo server paths to client paths (i.e. docker), if they are wrong, breakpoints don't stick

https://github.com/softwareCobbler/luceedebug?tab=readme-ov-file#vs-code-extension-configuration

@andreasRu
Copy link
Contributor Author

@zspitzer good point! I've tried using pathTransforms and the directive is being ignored when setting the breakpoints. The server sees the correct files, but the breakpoints aure unbound using wrong server path. Still looking into it. Please do not merge this PR until I make the changes. I'll notify you as soon I have gathered more informations. This PR is still faulty and need changes. Maybe I should close it and make a full new PR when ready?

@zspitzer
Copy link
Member

why close, just update it

@zspitzer
Copy link
Member

I did update that stuff myself, I'm happy to merge this and then you can do a follow up PR to improve it?

@andreasRu
Copy link
Contributor Author

andreasRu commented Jul 13, 2024

No need to. I have finished it yesterday, going to do a recheck because I did various changes. Going to update it until tomorrow

Another important issue is: I think the latest downloadable version doesn't notify the breakpoints correctly with pathTransform.
softwareCobbler/luceedebug#55

@andreasRu
Copy link
Contributor Author

Added and fixed stuff. I've merged the latest master into this PR branch. Now Linter failed with checking some other stuff (outside my changes). I was about to fix those, but didn't want to mix up changes coming from other merges. Letting this here for you to decide how to do it. Hope I didn't add additional work.

@zspitzer zspitzer merged commit a868a87 into lucee:master Jul 14, 2024
3 of 5 checks passed
@zspitzer
Copy link
Member

@andreasRu andreasRu deleted the lucee-debug-installation-guide branch July 14, 2024 10:24
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