better names #5

Merged

rozetko merged 2 commits from better-names into master 2 years ago

rozetko commented 2 years ago

Owner

This PR introduces lots of renamings that help better understand what's going on

Metric -> QueryConfig
MetricQuery -> DatasourceQuery
MetricResult -> DataTable (not sure it's the best name though, but it's better)
AbstractMetric -> DatasourceConnector
<datasource_name>Metric -> <datasource_name>Connector
other minor renamings

👍 1

rozetko added 2 commits 2 years ago

5e20b4de96 better names

087e001534 rename files

rozetko requested review from glitch4347 2 years ago

rozetko requested review from vargburz 2 years ago

rozetko merged commit bc6988b31b into master 2 years ago

rozetko deleted branch better-names 2 years ago

rozetko referenced this issue from a commit 2 years ago

Merge pull request 'better names' (#5) from better-names into master

Reviewers

glitch4347 was requested for review 2 years ago

vargburz was requested for review 2 years ago

The pull request has been merged as bc6988b31b.

Sign in to join this conversation.

No Label

No Milestone

No project

No Assignees

1 Participants

Notifications

Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.