Code formatting

GjjvdBurg · Sep 24, 2023 · 5b3b12d · 5b3b12d
1 parent 6efd518
commit 5b3b12d
Show file tree

Hide file tree

Showing 20 changed files with 12 additions and 36 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -16,7 +16,7 @@ jobs:
     runs-on: [ 'ubuntu-latest' ]
     strategy:
       matrix:
-        py: [ '3.8', '3.11' ]
+        py: [ '3.8', '3.11' ] # minimum required and latest stable
 
     steps:
       - name: Install Python ${{ matrix.py }}
@@ -27,12 +27,18 @@ jobs:
       - name: Checkout code
         uses: actions/checkout@v2
 
-      - name: Run unit test script
-        run: ./.github/scripts/test_p2r.sh
-        shell: bash
+      # NOTE: Keep versions in sync with .pre-commit-config.yaml
 
       - name: Run code quality tests (black)
         uses: psf/black@stable
+        with:
+          version: "23.3.0"
 
       - name: Run code quality tests (isort)
         uses: jamescurtin/isort-action@master
+        with:
+          isortVersion: "5.12.0"
+
+      - name: Run unit test script
+        run: ./.github/scripts/test_p2r.sh
+        shell: bash
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -1,6 +1,7 @@
+# NOTE: Keep versions in sync with Github Actions test.yml
 repos:
   - repo: https://github.com/psf/black
-    rev: 22.3.0
+    rev: 23.3.0
     hooks:
       - id: black
         language_version: python3

diff --git a/paper2remarkable/providers/acl.py b/paper2remarkable/providers/acl.py
@@ -16,15 +16,13 @@
 
 
 class ACLInformer(Informer):
-
     meta_date_key = "citation_publication_date"
 
     def _format_authors(self, soup_authors):
         return super()._format_authors(soup_authors, sep=" ", idx=-1)
 
 
 class ACL(Provider):
-
     re_abs_1 = "^https://www.aclweb.org/anthology/(?P<key>[0-9a-zA-Z\.\-]+)"
     re_abs_2 = "^https://(www.)?aclanthology.org/(?P<key>[0-9a-zA-Z\.\-]+)"
     re_pdf_1 = "^https://www.aclweb.org/anthology/(?P<key>[0-9a-zA-Z\.\-]*?)(v\d+)?.pdf"

diff --git a/paper2remarkable/providers/acm.py b/paper2remarkable/providers/acm.py
@@ -43,7 +43,6 @@ def _format_year(self, soup_date):
 
 
 class ACM(Provider):
-
     re_abs = "^https?://dl.acm.org/doi/(?P<doi>\d+\.\d+/\d+\.\d+)"
     re_pdf = "^https?://dl.acm.org/doi/pdf/(?P<doi>\d+\.\d+/\d+\.\d+)(\?download=true)?"
 

diff --git a/paper2remarkable/providers/arxiv.py b/paper2remarkable/providers/arxiv.py
@@ -29,7 +29,6 @@ class ArxivInformer(Informer):
 
 
 class Arxiv(Provider):
-
     re_abs_1 = "https?://arxiv.org/abs/\d{4}\.\d{4,5}(v\d+)?"
     re_pdf_1 = "https?://arxiv.org/pdf/\d{4}\.\d{4,5}(v\d+)?\.pdf"
 

diff --git a/paper2remarkable/providers/citeseerx.py b/paper2remarkable/providers/citeseerx.py
@@ -20,7 +20,6 @@
 
 
 class CiteSeerXInformer(Informer):
-
     meta_author_key = "citation_authors"
     meta_date_key = "citation_year"
 
@@ -30,7 +29,6 @@ def _format_authors(self, soup_authors):
 
 
 class CiteSeerX(Provider):
-
     re_abs = "^https?:\/\/citeseerx.ist.psu.edu(:443)?\/viewdoc\/summary\?doi=(?P<doi>[0-9\.]+)"
     re_pdf = "^https?:\/\/citeseerx.ist.psu.edu(:443)?\/viewdoc\/download(\;jsessionid=[A-Z0-9]+)?\?doi=(?P<doi>[0-9\.]+)&rep=rep1&type=pdf"
 

diff --git a/paper2remarkable/providers/cvf.py b/paper2remarkable/providers/cvf.py
@@ -19,12 +19,10 @@
 
 
 class CVFInformer(Informer):
-
     meta_date_key = "citation_publication_date"
 
 
 class CVF(Provider):
-
     re_abs = "^https?://openaccess.thecvf.com/content_([\w\d]+)/html/([\w\d\_\-]+).html$"
     re_pdf = "^https?://openaccess.thecvf.com/content_([\w\d]+)/papers/([\w\d\_\-]+).pdf$"
 

diff --git a/paper2remarkable/providers/eccc.py b/paper2remarkable/providers/eccc.py
@@ -64,7 +64,6 @@ def get_year(self, soup):
 
 
 class ECCC(Provider):
-
     re_abs = "https?://eccc.weizmann.ac.il/report/\d{4}/\d+/?$"
     re_pdf = "https?://eccc.weizmann.ac.il/report/\d{4}/\d+/download/?$"
 

diff --git a/paper2remarkable/providers/iacr.py b/paper2remarkable/providers/iacr.py
@@ -62,7 +62,6 @@ def get_year(self, soup):
 
 
 class IACR(Provider):
-
     re_abs = "https?://eprint.iacr.org/\d{4}/\d+$"
     re_pdf = "https?://eprint.iacr.org/\d{4}/\d+\.pdf$"
     re_ps = "https?://eprint.iacr.org/\d{4}/\d+\.ps$"

diff --git a/paper2remarkable/providers/jmlr.py b/paper2remarkable/providers/jmlr.py
@@ -18,7 +18,6 @@
 
 
 class JMLRInformer(Informer):
-
     meta_date_key = "citation_publication_date"
 
     def _format_authors(self, soup_authors):
@@ -29,7 +28,6 @@ def _format_authors(self, soup_authors):
 
 
 class JMLR(Provider):
-
     re_abs_1 = "https?://(www\.)?jmlr\.org/papers/v(?P<vol>\d+)/(?P<pid>\d{2}\-\d{3}).html$"
     re_pdf_1 = "https?://(www\.)?jmlr\.org/papers/volume(?P<vol>\d+)/(?P<pid>\d{2}\-\d{3})/(?P=pid).pdf$"
 

diff --git a/paper2remarkable/providers/nature.py b/paper2remarkable/providers/nature.py
@@ -16,15 +16,13 @@
 
 
 class NatureInformer(Informer):
-
     meta_date_key = "citation_online_date"
 
     def _format_authors(self, soup_authors):
         return super()._format_authors(soup_authors, sep=",", idx=0)
 
 
 class Nature(Provider):
-
     re_abs = "^https://www.nature.com/articles/s[a-z0-9\-]+$"
     re_pdf = "^https://www.nature.com/articles/s[a-z0-9\-]+\.pdf$"
 

diff --git a/paper2remarkable/providers/nber.py b/paper2remarkable/providers/nber.py
@@ -18,15 +18,13 @@
 
 
 class NBERInformer(Informer):
-
     meta_date_key = "citation_publication_date"
 
     def _format_authors(self, soup_authors, sep=" ", idx=0, op=None):
         return super()._format_authors(soup_authors, sep=" ", idx=-1, op=None)
 
 
 class NBER(Provider):
-
     re_abs = "https?://www\.nber\.org/papers/(?P<ref>[a-z0-9]+)$"
     re_pdf = "https?://www\.nber\.org/papers/(?P<ref>[a-z0-9]+)\.pdf$"
 

diff --git a/paper2remarkable/providers/openreview.py b/paper2remarkable/providers/openreview.py
@@ -20,7 +20,6 @@
 
 
 class OpenReviewInformer(Informer):
-
     meta_date_key = "citation_publication_date"
 
     def get_authors(self, soup):
@@ -56,7 +55,6 @@ def _format_authors(self, soup_authors):
 
 
 class OpenReview(Provider):
-
     re_abs = "https?://openreview.net/forum\?id=[A-Za-z0-9]+"
     re_pdf = "https?://openreview.net/pdf\?id=[A-Za-z0-9]+"
 

diff --git a/paper2remarkable/providers/pmlr.py b/paper2remarkable/providers/pmlr.py
@@ -16,15 +16,13 @@
 
 
 class PMLRInformer(Informer):
-
     meta_date_key = "citation_publication_date"
 
     def _format_authors(self, soup_authors):
         return super()._format_authors(soup_authors, sep=" ", idx=-1)
 
 
 class PMLR(Provider):
-
     re_abs_1 = "https?://proceedings.mlr.press/v\d+/[\w\-\w]+\d+.html"
     re_pdf_1 = "https?://proceedings.mlr.press/v\d+/[\w\-\w]+\d+.pdf"
 

diff --git a/paper2remarkable/providers/pubmed.py b/paper2remarkable/providers/pubmed.py
@@ -16,7 +16,6 @@
 
 
 class PubMedInformer(Informer):
-
     meta_date_key = "citation_publication_date"
     meta_author_key = "citation_author"
 
@@ -25,7 +24,6 @@ def _format_authors(self, soup_authors):
 
 
 class PubMed(Provider):
-
     re_abs = "https?://www.ncbi.nlm.nih.gov/pmc/articles/PMC\d+/?"
     re_pdf = (
         "https?://www.ncbi.nlm.nih.gov/pmc/articles/PMC\d+/pdf/nihms\d+\.pdf"

diff --git a/paper2remarkable/providers/sagepub.py b/paper2remarkable/providers/sagepub.py
@@ -16,7 +16,6 @@
 
 
 class SagePubInformer(Informer):
-
     meta_author_key = "dc.Creator"
     meta_title_key = "dc.Title"
     meta_date_key = "dc.Date"
@@ -29,7 +28,6 @@ def _format_year(self, soup_date):
 
 
 class SagePub(Provider):
-
     re_abs = "https?:\/\/journals\.sagepub\.com\/doi\/full\/\d{2}\.\d{4}\/\d+"
     re_pdf = "https?:\/\/journals\.sagepub\.com\/doi\/pdf\/\d{2}\.\d{4}\/\d+"
 

diff --git a/paper2remarkable/providers/science_direct.py b/paper2remarkable/providers/science_direct.py
@@ -29,7 +29,6 @@
 
 
 class ScienceDirectInformer(Informer):
-
     meta_date_key = "citation_publication_date"
 
     def get_authors(self, soup):
@@ -44,7 +43,6 @@ def get_authors(self, soup):
 
 
 class ScienceDirect(Provider):
-
     re_abs = (
         "https?:\/\/www.sciencedirect.com/science/article/pii/[A-Za-z0-9]+"
     )

diff --git a/paper2remarkable/providers/semantic_scholar.py b/paper2remarkable/providers/semantic_scholar.py
@@ -20,15 +20,13 @@
 
 
 class SemanticScholarInformer(Informer):
-
     meta_date_key = "citation_publication_date"
 
     def _format_authors(self, soup_authors):
         return super()._format_authors(soup_authors, sep=" ", idx=-1)
 
 
 class SemanticScholar(Provider):
-
     re_abs = (
         "https?:\/\/www.semanticscholar.org/paper/[A-Za-z0-9%\-]+/[0-9a-f]{40}"
     )

diff --git a/paper2remarkable/providers/springer.py b/paper2remarkable/providers/springer.py
@@ -20,7 +20,6 @@
 
 
 class SpringerInformer(Informer):
-
     meta_date_key = None
 
     def _format_authors(self, soup_authors):
@@ -36,7 +35,6 @@ def get_year(self, soup):
 
 
 class Springer(Provider):
-
     re_abs_1 = "https?:\/\/link.springer.com\/article\/10\.\d{4}\/[a-z0-9\-]+"
     re_abs_2 = "https?:\/\/link.springer.com\/chapter\/10\.\d{4}\/[a-z0-9\-]+"
     re_pdf = "https?:\/\/link\.springer\.com\/content\/pdf\/10\.\d{4}(%2F|\/)[a-z0-9\-\_]+\.pdf"

diff --git a/paper2remarkable/providers/tandfonline.py b/paper2remarkable/providers/tandfonline.py
@@ -31,7 +31,6 @@ def _format_year(self, soup_date):
 
 
 class TandFOnline(Provider):
-
     re_abs = "^https?://\w+.tandfonline.com/doi/(full|abs)/(?P<doi>\d+\.\d+/\w+\.\w+\.\w+)"
     re_pdf = "^https?://\w+.tandfonline.com/doi/(full|pdf)/(?P<doi>\d+\.\d+/\w+\.\w+\.\w+)"