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

Expose app root path to front-end #882

Merged
merged 6 commits into from
Sep 14, 2023
Merged

Expose app root path to front-end #882

merged 6 commits into from
Sep 14, 2023

Conversation

simon-johansson
Copy link
Contributor

@simon-johansson simon-johansson commented Sep 7, 2023

So that we can open the files in the users editor

@encore-cla
Copy link

encore-cla bot commented Sep 7, 2023

All committers have signed the CLA.

@github-actions
Copy link

github-actions bot commented Sep 7, 2023

Test Results

    2 files  +       1  224 suites  +66   5m 19s ⏱️ - 11m 6s
714 tests +   179  711 ✔️ +   189  3 💤 +3  0  - 4 
714 runs   - 1 885  711 ✔️  - 1 875  3 💤 +3  0  - 4 

Results for commit 2b5b058. ± Comparison against base commit 9166645.

This pull request removes 24 and adds 203 tests. Note that renamed tests count towards both.
encr.dev/cli/daemon/dash ‑ TestSourceContext
encr.dev/cli/daemon/dash ‑ TestSourceContext/0
encr.dev/cli/daemon/dash ‑ TestSourceContext/1
encr.dev/cli/daemon/dash ‑ TestSourceContext/2
encr.dev/cli/daemon/dash ‑ TestSourceContext/3
encr.dev/cli/daemon/dash ‑ TestSourceContext/4
encr.dev/cli/daemon/dash ‑ TestSourceContext/5
encr.dev/e2e-tests ‑ TestRun/encore_currentrequest
encr.dev/e2e-tests ‑ TestRun/et_override_user
encr.dev/e2e-tests ‑ TestRun/et_override_user_authdata
…
encore.dev/appruntime/apisdk/api ‑ TestDescGeneratesTrace
encore.dev/appruntime/apisdk/api ‑ TestDescGeneratesTrace/echo
encore.dev/appruntime/apisdk/api ‑ TestDescGeneratesTrace/invalid
encore.dev/appruntime/apisdk/api ‑ TestDescGeneratesTrace/raw
encore.dev/appruntime/apisdk/api ‑ TestDescGeneratesTrace/unauthenticated
encore.dev/appruntime/apisdk/api ‑ TestDesc_EndToEnd
encore.dev/appruntime/apisdk/api ‑ TestDesc_EndToEnd/echo
encore.dev/appruntime/apisdk/api ‑ TestDesc_EndToEnd/invalid
encore.dev/appruntime/apisdk/api ‑ TestDesc_EndToEnd/unauthenticated
encore.dev/appruntime/apisdk/api ‑ TestRawEndpointOverflow
…

♻️ This comment has been updated with latest results.

@simon-johansson simon-johansson enabled auto-merge (squash) September 8, 2023 12:11
@simon-johansson simon-johansson requested review from DomBlack and removed request for eandre September 8, 2023 12:12
This commit ports over the editor package from GitHub desktop from TypeScript into Go and gives us the ability to find editors on the user file system, list them in the UI and then open files in the UI
pkg/editors/launch.go Outdated Show resolved Hide resolved
pkg/editors/launch.go Outdated Show resolved Hide resolved
pkg/editors/launch.go Outdated Show resolved Hide resolved
pkg/editors/utils.go Outdated Show resolved Hide resolved
pkg/editors/launch.go Show resolved Hide resolved
pkg/editors/launch.go Show resolved Hide resolved
@DomBlack DomBlack requested review from eandre and removed request for DomBlack September 8, 2023 17:19
cli/daemon/dash/dash.go Outdated Show resolved Hide resolved
pkg/editors/launch_nonwindows.go Outdated Show resolved Hide resolved
pkg/editors/lookup.go Outdated Show resolved Hide resolved
pkg/editors/lookup_darwin.go Outdated Show resolved Hide resolved
@simon-johansson simon-johansson merged commit 99069d9 into main Sep 14, 2023
5 checks passed
@simon-johansson simon-johansson deleted the app-path-websocket branch September 14, 2023 11:13
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.

3 participants