add comments

Author: bitnik

binderhub/repoproviders.py

@@ -227,6 +227,9 @@ def get_resolved_ref(self):
     async def get_resolved_spec(self):
         if not hasattr(self, 'record_id'):
             self.record_id = await self.get_resolved_ref()
+        # zenodo registers a DOI which represents all versions.
+        # and it always resolves to latest version
+        # for that case, we have to replace it with record id
         resolved_spec = self.spec.split("zenodo")[0] + "zenodo." + self.record_id
         return resolved_spec
 

binderhub/tests/test_repoproviders.py

@@ -43,6 +43,8 @@ def test_spec_processing(spec, raw_user, raw_repo, raw_ref):
      '3242074',
      'https://doi.org/10.5281/zenodo.3242074',
      'zenodo-3242074'],
+    # 10.5281/zenodo.705645 -> This DOI represents all versions, and will always resolve to the latest one
+    # for now it is 3242074
     ['10.5281/zenodo.3242073',
      '10.5281/zenodo.3242074',
      '3242074',