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

✨ NEW: Transformer for removing dict and list keywords #4

Merged
merged 1 commit into from
Mar 11, 2022

Conversation

mbercx
Copy link
Member

@mbercx mbercx commented Mar 10, 2022

We recently adapted the constructors of the Dict and List nodes so
they no longer require the dict and list keywords, see:

aiidateam/aiida-core#5165

Here we add a transformer that automatically removes these keywords from
Dict and List constructors if present.

@mbercx mbercx requested a review from chrisjsewell March 10, 2022 12:33
@mbercx mbercx added the pr/blocked PR is blocked by another that needs to be merged first label Mar 10, 2022
@mbercx
Copy link
Member Author

mbercx commented Mar 10, 2022

Still blocked by #3

Ready for review!

@mbercx mbercx removed the pr/blocked PR is blocked by another that needs to be merged first label Mar 10, 2022
@mbercx mbercx force-pushed the new/dict-list-keywords branch 3 times, most recently from df5b293 to f8be454 Compare March 10, 2022 22:59
We recently adapted the constructors of the `Dict` and `List` nodes so
they no longer require the `dict` and `list` keywords, see:

aiidateam/aiida-core#5165

Here we add a transformer that automatically removes these keywords from
`Dict` and `List` constructors if present.
@mbercx mbercx force-pushed the new/dict-list-keywords branch from f8be454 to a0923a2 Compare March 10, 2022 23:01
@sphuber sphuber self-requested a review March 11, 2022 16:53
Copy link

@sphuber sphuber left a comment

Choose a reason for hiding this comment

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

Would have maybe added a test with a positional argument and another keyword in the same constructor for extra robustness but this can always be added later. Let's get this show on the road

@mbercx mbercx merged commit 91bcd98 into aiidateam:main Mar 11, 2022
@mbercx mbercx deleted the new/dict-list-keywords branch March 11, 2022 16:55
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.

2 participants