Skip to content

Migrate the type and process type string of built in calculation plugins#2308

Merged
giovannipizzi merged 1 commit into
aiidateam:provenance_redesignfrom
sphuber:fix_2306_migrate_builtin_calculation_entry_points
Dec 6, 2018
Merged

Migrate the type and process type string of built in calculation plugins#2308
giovannipizzi merged 1 commit into
aiidateam:provenance_redesignfrom
sphuber:fix_2306_migrate_builtin_calculation_entry_points

Conversation

@sphuber
Copy link
Copy Markdown
Contributor

@sphuber sphuber commented Dec 6, 2018

Fixes #2306

The built in calculation entry points were recently moved and renamed
but no migration was put in.

This requires renaming the node types, process types and the input plugin
attribute of code instances.

@sphuber sphuber requested a review from giovannipizzi December 6, 2018 14:27
@sphuber sphuber force-pushed the fix_2306_migrate_builtin_calculation_entry_points branch from 5a9d7c3 to d7d343c Compare December 6, 2018 14:34
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Missing ;?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Missing ;?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Needs a Join on Db_Dbnode to do the same as SQLA

@giovannipizzi
Copy link
Copy Markdown
Member

Also, there is a pre-commit thing to fix

@sphuber sphuber force-pushed the fix_2306_migrate_builtin_calculation_entry_points branch 2 times, most recently from 7c0d05c to 95ad6b2 Compare December 6, 2018 17:13
The built in calculation entry points were recently moved and renamed
but no migration was put in.

This requires renaming the node types, process types and the input plugin
attribute of code instances.
@sphuber sphuber force-pushed the fix_2306_migrate_builtin_calculation_entry_points branch from 95ad6b2 to c6e5381 Compare December 6, 2018 18:10
@giovannipizzi giovannipizzi merged commit d0a9ad9 into aiidateam:provenance_redesign Dec 6, 2018
@sphuber sphuber deleted the fix_2306_migrate_builtin_calculation_entry_points branch December 7, 2018 10:18
@coveralls
Copy link
Copy Markdown

coveralls commented Dec 8, 2018

Coverage Status

Coverage decreased (-0.03%) to 68.904% when pulling c6e5381 on sphuber:fix_2306_migrate_builtin_calculation_entry_points into d4dce2d on aiidateam:provenance_redesign.

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.

3 participants