Skip to content

キャピタリゼーションの変更など - #14

Closed
nagata-yoshiteru with Copilot wants to merge 1 commit into
feature/fix-capitalizefrom
copilot/sub-pr-13
Closed

キャピタリゼーションの変更など#14
nagata-yoshiteru with Copilot wants to merge 1 commit into
feature/fix-capitalizefrom
copilot/sub-pr-13

Conversation

Copilot AI commented Dec 2, 2025

Copy link
Copy Markdown

プロパティ名をPascalCaseからcamelCaseに変更し、命名規則の一貫性を確保。

変更内容

  • プロパティ名のcamelCase化 (10項目):
    • BaseIncrementbaseIncrement
    • DecimalPlacesdecimalPlaces
    • LossyCompressionRatelossyCompressionRate
    • PrecisionBytesprecisionBytes
    • RangeMin/RangeMaxrangeMin/rangeMax
    • StripTrailingZerosstripTrailingZeros
    • UseBaseIncrementuseBaseIncrement
    • UseFFTCompressionuseFFTCompression
    • UseRunLengthuseRunLength

命名規則

種別 規則
クラス/メッセージ PascalCase RealWorldDataset
列挙型 PascalCase VariableScaleTypeEnumeration
プロパティ camelCase variableScaleType

対象ファイル

  • dbp_schema.proto / dbp_schema.ja.proto
  • dbp-schema.jsonld / dbp-schema.ja.jsonld
  • README.md

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Update variable capitalization for consistency キャピタリゼーションの変更など Dec 2, 2025
@nagata-yoshiteru

Copy link
Copy Markdown
Member

@copilot ちゃんとすべての変更箇所が正しくルールに沿って変更されてた?

Copilot AI commented Dec 2, 2025

Copy link
Copy Markdown
Author

@copilot ちゃんとすべての変更箇所が正しくルールに沿って変更されてた?

はい、すべての変更箇所を確認しました。

検証結果:

プロパティ名(10項目): すべてcamelCaseに変更済み

  • baseIncrement, decimalPlaces, lossyCompressionRate, precisionBytes, rangeMin, rangeMax, stripTrailingZeros, useBaseIncrement, useFFTCompression, useRunLength

対象ファイル: すべて正しく更新済み

  • dbp_schema.proto / dbp_schema.ja.proto
  • dbp-schema.jsonld / dbp-schema.ja.jsonld
  • README.md

命名規則の一貫性:

  • クラス/メッセージ名: PascalCase維持(例: RealWorldDataset
  • 列挙型名: PascalCase維持(例: VariableScaleTypeEnumeration
  • プロパティ名: camelCase変更済み

すべて正しくルールに沿って変更されています。

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.

2 participants