starred/windmill
1
0
Fork 0
mirror of https://github.com/windmill-labs/windmill.git synced 2026-08-20 16:02:19 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: include workspacedependencies in default git sync include_type (#8852)

Browse Source 
* fix: include workspacedependencies in default git sync include_type

Workspace dependencies were not synced by default because the default
include_type arrays did not contain 'workspacedependencies'. This meant
users had to manually add the type to their git sync config.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: add missing folder to new-format fallback and align all defaults

- Add 'folder' to the new-format repo fallback default include_type
  (was missing unlike the other 4 locations)
- Add 'workspacedependencies' to legacy defaultTypes fallback
- Align GitSyncFilterSettings prop default with context defaults

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Ruben Fiszel
2026-04-16 07:29:38 -07:00
committed by GitHub
parent b1aeb33ade
commit fc49a8fed6
2 changed files with 18 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
frontend/src/lib/components/git_sync/GitSyncContext.svelte.ts
+11 -5
View File
@@ -194,7 +194,7 @@ export function createGitSyncContext(workspace: string) {
include_path: ['f/**'],
exclude_path: [],
extra_include_path: [],
include_type: ['script', 'flow', 'app', 'folder']
include_type: ['script', 'flow', 'app', 'folder', 'workspacedependencies']
},
exclude_types_override: [],
legacyImported: false,
@@ -408,7 +408,7 @@ export function createGitSyncContext(workspace: string) {
const defaultTypes: GitSyncObjectType[] =
workspaceLegacyIncludeType.length > 0
? [...workspaceLegacyIncludeType]
: ['script', 'flow', 'app', 'folder']
: ['script', 'flow', 'app', 'folder', 'workspacedependencies']
let repoSettings: SettingsObject
if (isRepoLegacy) {
@@ -433,7 +433,13 @@ export function createGitSyncContext(workspace: string) {
include_path: repo.settings?.include_path ?? ['f/**'],
exclude_path: repo.settings?.exclude_path ?? [],
extra_include_path: repo.settings?.extra_include_path ?? [],
include_type: repo.settings?.include_type ?? ['script', 'flow', 'app']
include_type: repo.settings?.include_type ?? [
'script',
'flow',
'app',
'folder',
'workspacedependencies'
]
}
}
@@ -662,7 +668,7 @@ export function createGitSyncContext(workspace: string) {
include_path: ['f/**'],
exclude_path: [],
extra_include_path: [],
include_type: ['script', 'flow', 'app', 'folder']
include_type: ['script', 'flow', 'app', 'folder', 'workspacedependencies']
},
exclude_types_override: [],
legacyImported: false,
@@ -689,7 +695,7 @@ export function createGitSyncContext(workspace: string) {
include_path: ['f/**'],
exclude_path: [],
extra_include_path: [],
include_type: ['script', 'flow', 'app', 'folder']
include_type: ['script', 'flow', 'app', 'folder', 'workspacedependencies']
},
exclude_types_override: [],
legacyImported: false,
frontend/src/lib/components/workspaceSettings/GitSyncFilterSettings.svelte
+7 -1
View File
@@ -30,7 +30,13 @@
let {
git_repo_resource_path = $bindable(''),
include_path = $bindable(['f/**']),
include_type = $bindable(['script', 'flow', 'app', 'folder'] as GitSyncObjectType[]),
include_type = $bindable([
'script',
'flow',
'app',
'folder',
'workspacedependencies'
] as GitSyncObjectType[]),
exclude_types_override = $bindable([] as GitSyncObjectType[]),
isLegacyRepo = false,
excludes = $bindable([] as string[]),