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

YAN-853 DataDROP type and add data components guide #101

Merged
merged 10 commits into from
Feb 7, 2022
Merged

Conversation

calgray
Copy link
Collaborator

@calgray calgray commented Feb 2, 2022

Added a DataDROP class layer between AbstractionDROP and all the data drop classes similar to what AppDROP provides. This serves as a documented abstract interface for what DataDROP developers need to implement and override.

Changes:

  • Moved DataDROP related methods out of AbstractDROP
  • Ordered DataDROPs in source

@coveralls
Copy link

coveralls commented Feb 2, 2022

Coverage Status

Coverage decreased (-2.2%) to 77.033% when pulling 7900014 on YAN-853 into e741689 on master.

@calgray calgray changed the title YAN-853 DataDROP YAN-853 DataDROP type and add data components guide Feb 7, 2022
Base automatically changed from devall-docker to master February 7, 2022 12:34
@awicenec awicenec merged commit 07654d4 into master Feb 7, 2022
@awicenec awicenec deleted the YAN-853 branch February 7, 2022 13:36
awicenec added a commit that referenced this pull request May 19, 2022
YAN-853 DataDROP type and add data components guide
pritchardn pushed a commit that referenced this pull request May 20, 2022
YAN-853 DataDROP type and add data components guide
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