Skip to content

Conversation

Wroud
Copy link
Member

@Wroud Wroud commented Sep 2, 2025

closes dbeaver/pro#6662

@Wroud Wroud requested a review from devnaumov September 2, 2025 08:31
@Wroud Wroud self-assigned this Sep 2, 2025
@@ -4,7 +4,7 @@
"type": "module",
"private": true,
"workspaces": [
"../common-react",
"./common-react",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we add ./common-typescript also?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, it's linked via common-react

@@ -2,6 +2,6 @@
<?pde version="3.8"?>
<target name="org.jkiss.cloudbeaver.tp">
<locations>
<location path="../../dbeaver-workspace/dependencies/plugins" type="Directory"/>
<location path="../../../dbeaver-workspace/dependencies/plugins" type="Directory"/>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you tell why the backend-related file changed?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This folder was moved from the repo's root to cloudbeaver/.vscode, so it needs to be updated.
This is the configuration needed to debug the server in VS Code (it will not affect the build or IDEA)

sergeyteleshev

This comment was marked as resolved.

@Wroud Wroud merged commit f41b9d2 into devel Sep 2, 2025
9 of 10 checks passed
@Wroud Wroud deleted the dbeaver/pro#6662/cleanup branch September 2, 2025 10:41
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.

5 participants