Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sql: rename Field to Column. #100

Merged
merged 1 commit into from
Feb 9, 2017
Merged

sql: rename Field to Column. #100

merged 1 commit into from
Feb 9, 2017

Conversation

smola
Copy link
Contributor

@smola smola commented Jan 27, 2017

A Field would mean a cell in a row, but our previous
Field type is actually a column definition in a schema,
so Column is a more previse name.

A Field would mean a cell in a row, but our previous
Field type is actually a column definition in a schema,
so Column is a more previse name.
@smola smola added the enhancement New feature or request label Jan 27, 2017
@smola smola added this to the v0.5.0 milestone Jan 27, 2017
@codecov-io
Copy link

Current coverage is 61.01% (diff: 94.11%)

Merging #100 into master will not change coverage

@@             master       #100   diff @@
==========================================
  Files            40         40          
  Lines          1652       1652          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits           1008       1008          
  Misses          585        585          
  Partials         59         59          

Powered by Codecov. Last update 761ba5e...0834392

@smola smola merged commit f2951ee into src-d:master Feb 9, 2017
@smola smola deleted the field-to-column branch February 9, 2017 17:07
dpordomingo pushed a commit to dpordomingo/gitbase that referenced this pull request Mar 10, 2017
A Field would mean a cell in a row, but our previous
Field type is actually a column definition in a schema,
so Column is a more previse name.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants