Skip to content
Snippets Groups Projects
Unverified Commit 621b77b2 authored by Jamie Magee's avatar Jamie Magee Committed by GitHub
Browse files

refactor(datasource): migrate to class based datasource (#6747)


* refactor(datasource): migrate to class based datasource

    A small experiment into what OOP/class based datasources might look like. Picked Cdnjs as it's the smallest & simplest.

    With this approach we can share common logic, like error handling, rate limiting, etc. between different datasources, instead of having to reimplement it each time we write a new datasource. Currently there's nothing shared, as it's only 1 datasource, but the interesting stuff will come with the 2nd datasource

* Apply suggestions from code review

Co-authored-by: default avatarMichael Kriese <michael.kriese@visualon.de>

* remove renaming ClojureDatasource to datasource in tests

Co-authored-by: default avatarRhys Arkins <rhys@arkins.net>
Co-authored-by: default avatarMichael Kriese <michael.kriese@visualon.de>
parent b5189746
No related branches found
No related tags found
No related merge requests found
Showing
with 1191 additions and 85 deletions
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment