global: add redirection from legacy recids #199
Triggered via pull request
September 19, 2024 13:23
kpsherva
Status
Failure
Total duration
5m 34s
Artifacts
–
Annotations
13 errors and 1 warning
|
Tests (site, 3.9, postgresql14, opensearch2, redis, rabbitmq, 18):
site/cds_rdm/minters.py#L1
pydocstyle-check
/home/runner/work/cds-rdm/cds-rdm/site/cds_rdm/minters.py:1 at module level:
D100: Missing docstring in public module
|
|
Tests (site, 3.9, postgresql14, opensearch2, redis, rabbitmq, 18):
site/cds_rdm/resolver.py#L1
pydocstyle-check
/home/runner/work/cds-rdm/cds-rdm/site/cds_rdm/resolver.py:1 at module level:
D100: Missing docstring in public module
|
|
Tests (site, 3.9, postgresql14, opensearch2, redis, rabbitmq, 18):
site/cds_rdm/views.py#L1
isort-check
"""Additional views."""
from flask import Blueprint, redirect
+from invenio_rdm_records.resources.urls import record_url_for
from cds_rdm.resolver import get_pid_by_legacy_recid
-from invenio_rdm_records.resources.urls import record_url_for
def legacy_redirect(id):
|
|
Tests (site, 3.9, postgresql14, opensearch2, redis, rabbitmq, 18):
site/cds_rdm/views.py#L1
pydocstyle-check
/home/runner/work/cds-rdm/cds-rdm/site/cds_rdm/views.py:17 in public function `legacy_redirect`:
D103: Missing docstring in public function
|
|
Tests (site, 3.9, postgresql14, opensearch2, redis, rabbitmq, 18):
site/cds_rdm/views.py#L1
Black format check
--- /home/runner/work/cds-rdm/cds-rdm/site/cds_rdm/views.py 2024-09-19 13:23:39.260964+00:00
+++ /home/runner/work/cds-rdm/cds-rdm/site/cds_rdm/views.py 2024-09-19 13:28:20.380891+00:00
@@ -16,10 +16,12 @@
def legacy_redirect(id):
pid = get_pid_by_legacy_recid(id)
url_path = record_url_for(pid_value=pid)
return redirect(url_path)
+
+
#
# Registration
#
def create_blueprint(app):
"""Register blueprint routes on app."""
|
|
Tests (site, 3.9, postgresql14, opensearch2, redis, rabbitmq, 18)
Process completed with exit code 1.
|
|
Tests (site, 3.12, postgresql14, opensearch2, redis, rabbitmq, 18)
The job was canceled because "site_3_9_postgresql14_ope" failed.
|
|
Tests (site, 3.12, postgresql14, opensearch2, redis, rabbitmq, 18):
site/cds_rdm/minters.py#L1
pydocstyle-check
/home/runner/work/cds-rdm/cds-rdm/site/cds_rdm/minters.py:1 at module level:
D100: Missing docstring in public module
|
|
Tests (site, 3.12, postgresql14, opensearch2, redis, rabbitmq, 18):
site/cds_rdm/resolver.py#L1
pydocstyle-check
/home/runner/work/cds-rdm/cds-rdm/site/cds_rdm/resolver.py:1 at module level:
D100: Missing docstring in public module
|
|
Tests (site, 3.12, postgresql14, opensearch2, redis, rabbitmq, 18):
site/cds_rdm/views.py#L1
isort-check
"""Additional views."""
from flask import Blueprint, redirect
+from invenio_rdm_records.resources.urls import record_url_for
from cds_rdm.resolver import get_pid_by_legacy_recid
-from invenio_rdm_records.resources.urls import record_url_for
def legacy_redirect(id):
|
|
Tests (site, 3.12, postgresql14, opensearch2, redis, rabbitmq, 18):
site/cds_rdm/views.py#L1
pydocstyle-check
/home/runner/work/cds-rdm/cds-rdm/site/cds_rdm/views.py:17 in public function `legacy_redirect`:
D103: Missing docstring in public function
|
|
Tests (site, 3.12, postgresql14, opensearch2, redis, rabbitmq, 18):
site/cds_rdm/views.py#L1
Black format check
--- /home/runner/work/cds-rdm/cds-rdm/site/cds_rdm/views.py 2024-09-19 13:23:36.537455+00:00
+++ /home/runner/work/cds-rdm/cds-rdm/site/cds_rdm/views.py 2024-09-19 13:28:40.311161+00:00
@@ -16,10 +16,12 @@
def legacy_redirect(id):
pid = get_pid_by_legacy_recid(id)
url_path = record_url_for(pid_value=pid)
return redirect(url_path)
+
+
#
# Registration
#
def create_blueprint(app):
"""Register blueprint routes on app."""
|
|
Tests (site, 3.12, postgresql14, opensearch2, redis, rabbitmq, 18)
The operation was canceled.
|
|
Tests (site, 3.9, postgresql14, opensearch2, redis, rabbitmq, 18)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|