Skip to content

Feat/godot localization settings#178

Merged
GeWuYou merged 3 commits into
mainfrom
feat/godot-localization-settings
Apr 5, 2026
Merged

Feat/godot localization settings#178
GeWuYou merged 3 commits into
mainfrom
feat/godot-localization-settings

Conversation

@GeWuYou

@GeWuYou GeWuYou commented Apr 5, 2026

Copy link
Copy Markdown
Owner

No description provided.

GeWuYou added 2 commits April 5, 2026 19:54
- 新增LocalizationMap类实现用户语言到Godot locale和框架语言码的映射
- 创建GodotLocalizationSettings类同步应用本地化设置到Godot引擎和GFramework框架
- 添加测试项目配置文件GFramework.Game.Tests.csproj
- 实现本地化设置的单元测试验证语言同步功能
- 新增 Godot 设置模块概述和核心类说明
- 添加音频设置系统详细文档包括 AudioBusMap 和 GodotAudioSettings
- 添加图形设置系统文档包括 GodotGraphicsSettings 实现
- 添加本地化设置系统文档包括 LocalizationMap 和 GodotLocalizationSettings
- 提供完整的架构设计图和使用示例代码
- 添加 API 详细说明和技术实现细节
- 包含最佳实践、性能考虑和故障排除指南

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Sorry @GeWuYou, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@deepsource-io

deepsource-io Bot commented Apr 5, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in a22e522...e84af15 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
C# Apr 5, 2026 12:08p.m. Review ↗
Secrets Apr 5, 2026 12:08p.m. Review ↗

- 在 GodotLocalizationSettingsTests 中添加核心本地化接口引用
- 为 GodotLocalizationSettings 添加架构依赖项
- 配置 Godot 全局命名空间以便更简单的 API 访问
- 更新测试文件中的设置数据抽象层依赖项
- 简化 Godot 项目中的全局 using 指令结构
@GeWuYou GeWuYou merged commit 658a36f into main Apr 5, 2026
9 checks passed
@GeWuYou GeWuYou deleted the feat/godot-localization-settings branch April 5, 2026 12:21
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.

1 participant