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

10 base multi adaptor class #26

Closed
wants to merge 35 commits into from
Closed

Conversation

EddyCMWF
Copy link
Contributor

Multi adaptor class.
see #10 for more details

JamesVarndell and others added 30 commits July 10, 2023 16:37
* Refactor to create a more maintainable structure

* Renames adaptors back to original names for compatibility with the rest of the stack

* Adds comments around blocks to be deprecated

* Renames refereance to removed AbstractAdaptor class

* Renames adaptor classes back to original names

* Moved AbstractCdsAdaptor

* Removes AbstractCdsAdaptor from top level (moved to cds.py)

* updating actions

* QA tweaks

* Rverts naming of base abstract adaptor class

* Renames MARS and URL adaptors

---------

Co-authored-by: EddyCMWF <Edward.Comyn-Platt@ecmwf.int>
@EddyCMWF EddyCMWF requested a review from alexamici July 28, 2023 13:00
@FussyDuck
Copy link

FussyDuck commented Jul 28, 2023

CLA assistant check
All committers have signed the CLA.

@EddyCMWF EddyCMWF closed this Jul 28, 2023
@EddyCMWF EddyCMWF deleted the 10-base-multi-adaptor-class-main branch August 10, 2023 13:24
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