Skip to content

feat: optimize platform module config logic#1129

Merged
SparkYuan merged 2 commits intoKusionStack:mainfrom
SparkYuan:platform-config
May 21, 2024
Merged

feat: optimize platform module config logic#1129
SparkYuan merged 2 commits intoKusionStack:mainfrom
SparkYuan:platform-config

Conversation

@SparkYuan
Copy link
Member

What type of PR is this?

/kind feature

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?


Additional documentation e.g., design docs, usage docs, etc.:


SparkYuan added 2 commits May 21, 2024 17:02
- load workspace module config with module name
- ignore modules only exist in the workspace config
- optimize some err logs in callModules
@SparkYuan SparkYuan enabled auto-merge (squash) May 21, 2024 09:14
Copy link
Contributor

@liu-hm19 liu-hm19 left a comment

Choose a reason for hiding this comment

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

lgtm

@SparkYuan SparkYuan merged commit bd90c5a into KusionStack:main May 21, 2024
@github-actions github-actions bot locked and limited conversation to collaborators May 21, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants