Skip to content

feat: swap name and data parameters of Column#105

Merged
lars-reimann merged 3 commits intomainfrom
column_name_first
Mar 27, 2023
Merged

feat: swap name and data parameters of Column#105
lars-reimann merged 3 commits intomainfrom
column_name_first

Conversation

@lars-reimann
Copy link
Member

@lars-reimann lars-reimann commented Mar 27, 2023

Summary of Changes

The order of parameters of the __init__ method of Column is now

  1. name (was 2.)
  2. data (was 1.)
  3. type_.

Having the name first is more readable since the data can be quite long and highlights that the name acts as a key and the data as a value.

@lars-reimann lars-reimann requested a review from a team as a code owner March 27, 2023 20:11
@lars-reimann
Copy link
Member Author

lars-reimann commented Mar 27, 2023

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ PYTHON black 27 0 0 1.02s
✅ PYTHON flake8 27 0 0.58s
✅ PYTHON isort 27 0 0 0.25s
✅ PYTHON mypy 27 0 1.94s
✅ PYTHON pylint 27 0 3.44s
✅ REPOSITORY git_diff yes no 0.02s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

@codecov
Copy link

codecov bot commented Mar 27, 2023

Codecov Report

Merging #105 (cbf481e) into main (20aaf5e) will not change coverage.
The diff coverage is 100.00%.

❗ Current head cbf481e differs from pull request most recent head c773410. Consider uploading reports for the commit c773410 to get more accurate results

@@           Coverage Diff           @@
##             main     #105   +/-   ##
=======================================
  Coverage   92.06%   92.06%           
=======================================
  Files          36       36           
  Lines        1223     1223           
=======================================
  Hits         1126     1126           
  Misses         97       97           
Impacted Files Coverage Δ
src/safeds/data/tabular/containers/_column.py 90.14% <100.00%> (ø)
src/safeds/data/tabular/containers/_table.py 91.34% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@lars-reimann lars-reimann merged commit c2f8da5 into main Mar 27, 2023
@lars-reimann lars-reimann deleted the column_name_first branch March 27, 2023 20:20
lars-reimann pushed a commit that referenced this pull request Mar 29, 2023
## [0.7.0](v0.6.0...v0.7.0) (2023-03-29)

### Features

* `sort_rows` of a `Table` ([#104](#104)) ([20aaf5e](20aaf5e)), closes [#14](#14)
* add `_file` suffix to methods interacting with files ([#103](#103)) ([ec011e4](ec011e4))
* improve transformers for tabular data ([#108](#108)) ([b18a06d](b18a06d)), closes [#61](#61) [#90](#90)
* remove `OrdinalEncoder` ([#107](#107)) ([b92bba5](b92bba5)), closes [#61](#61)
* specify features and target when creating a `TaggedTable` ([#114](#114)) ([95e1fc7](95e1fc7)), closes [#27](#27)
* swap `name` and `data` parameters of `Column` ([#105](#105)) ([c2f8da5](c2f8da5))
@lars-reimann
Copy link
Member Author

🎉 This PR is included in version 0.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@lars-reimann lars-reimann added the released Included in a release label Mar 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released Included in a release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant