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

Remove old name "param_results" which was kept to keep the API for a while #537

Conversation

@uvchik
Copy link
Member

commented Nov 12, 2018

The old name was kept with a warning but for version v0.3.0 the old name will be removed.

Remove old name
The old name was kept due to compatibility reasons.

@uvchik uvchik added the API label Nov 12, 2018

@uvchik uvchik added this to the v0.3.0 milestone Nov 12, 2018

@uvchik uvchik self-assigned this Nov 12, 2018

uvchik added some commits Dec 11, 2018

Merge branch 'releases/v0_3_0' of github.com:oemof/oemof into feature…
…s/remove_param_results_for_v0_3_api_change_renamed

@uvchik uvchik requested a review from henhuy Dec 11, 2018

@uvchik uvchik added this to To do in Release of v0.3.0 via automation Dec 11, 2018

@coveralls

This comment has been minimized.

Copy link

commented Dec 11, 2018

Coverage Status

Coverage increased (+0.3%) to 88.322% when pulling 73258ad on features/remove_param_results_for_v0_3_api_change_renamed into 7d02b38 on releases/v0_3_0.

1 similar comment
@coveralls

This comment has been minimized.

Copy link

commented Dec 11, 2018

Coverage Status

Coverage increased (+0.3%) to 88.322% when pulling 73258ad on features/remove_param_results_for_v0_3_api_change_renamed into 7d02b38 on releases/v0_3_0.

Release of v0.3.0 automation moved this from To do to Progress Dec 12, 2018

@henhuy

henhuy approved these changes Dec 12, 2018

Copy link
Contributor

left a comment

okay! thanks

@uvchik uvchik merged commit 405a67b into releases/v0_3_0 Dec 12, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.3%) to 88.322%
Details

Release of v0.3.0 automation moved this from Progress to Done Dec 12, 2018

@uvchik uvchik deleted the features/remove_param_results_for_v0_3_api_change_renamed branch Dec 12, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
3 participants
You can’t perform that action at this time.