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

Rename ParameterData to Dict #2530

Merged

Conversation

sphuber
Copy link
Contributor

@sphuber sphuber commented Feb 25, 2019

Fixes #2398

@coveralls
Copy link

coveralls commented Feb 25, 2019

Coverage Status

Coverage decreased (-0.03%) to 69.752% when pulling c69f1b6 on sphuber:fix_2398_parameter_data_to_dict into 32be995 on aiidateam:provenance_redesign.

@sphuber sphuber force-pushed the fix_2398_parameter_data_to_dict branch from 32b5b78 to c026b20 Compare February 26, 2019 06:57
@sphuber
Copy link
Contributor Author

sphuber commented Feb 26, 2019

@giovannipizzi this one is also ready for review. Don't worry about the test failing, that is just pre-commit yapf error after rebase.

@sphuber sphuber force-pushed the fix_2398_parameter_data_to_dict branch 2 times, most recently from d0d100c to 3d001e7 Compare February 26, 2019 08:35
@sphuber sphuber force-pushed the fix_2398_parameter_data_to_dict branch from 3d001e7 to c69f1b6 Compare February 26, 2019 10:19
Copy link
Member

@giovannipizzi giovannipizzi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Goodbye ParameterData... you'll always be dear to my heart...

@sphuber sphuber merged commit 85fc1c4 into aiidateam:provenance_redesign Feb 26, 2019
@sphuber
Copy link
Contributor Author

sphuber commented Feb 26, 2019

We'll have a proper funeral once we release the beta, we'll celebrate the beautiful life it has had

@sphuber sphuber deleted the fix_2398_parameter_data_to_dict branch February 26, 2019 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants