microsoft / microsoft/flint-chart

Use `camelCase` instead of `kebab_case` in spec for consistency

Open
#102 4 comments 0 reactions 1 assignee View on GitHub

@Chenglong-MS is already working on this.

Since Aug 17, 2026.

Dominant language
TypeScript
Stars
4.2k
Forks
235
Avg merge
17h 45m
Merged PRs (30d)
11

Description

Currently, the root properties of the spec use kebab_case for properties e.g. semantic_types and chart_spec while everywhere else the spec uses camelCase for properties e.g. chartType. This is inconsistent not only within the schema, but also with the usual convention for JavaScript/TypeScript which is to use camelCase (I understand kebab_case is the standard in e.g. python), but having both is definitively odd and confusing.

{
-  "semantic_types": {
+  "semanticTypes": {
     "count": "Quantity",
     "percentage": "Percentage",
     "region": "Category"
   },
-  "chart_spec": {
+  "chartSpec": {
     "chartType": "Scatter Plot",
     "encodings": {
       "x": "count",
       "y": "percentage",
       "color": "region"
     }
   },
   "data": {}
 }

Maybe I am missing something, there must be a valid reason? Additionally, we could maybe rename chartSpec to simply spec?

In any case, standardizing this to use camelCase for all properties would be more consistent and standard.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.