Skip to content

rename Data/EffectDeclaration' to Data/EffectDeclaration #1652

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

Merged
merged 1 commit into from
Aug 15, 2020

Conversation

aryairani
Copy link
Contributor

I also moved the EffectDeclaration declaration closer to the top of the file.

@aryairani aryairani changed the title rename Data/EffectDeclaration' to Data/EffectDeclaration, etc. rename Data/EffectDeclaration' to Data/EffectDeclaration Aug 12, 2020
@aryairani aryairani requested a review from runarorama August 13, 2020 03:04
@aryairani
Copy link
Contributor Author

FYSA @dolio

@dolio
Copy link
Contributor

dolio commented Aug 13, 2020

Makes sense to me.

@aryairani aryairani requested review from pchiusano and dolio August 13, 2020 19:14
Copy link
Member

@pchiusano pchiusano left a comment

Choose a reason for hiding this comment

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

😎

@pchiusano pchiusano merged commit 1cf21b8 into trunk Aug 15, 2020
@pchiusano pchiusano deleted the topic/datadecl-cleanup branch August 15, 2020 18:12
@pchiusano pchiusano mentioned this pull request May 11, 2021
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