Skip to content

add date extractor#10

Merged
tschaffter merged 1 commit intonlpsandbox:developfrom
ymnliu:develop
Nov 27, 2020
Merged

add date extractor#10
tschaffter merged 1 commit intonlpsandbox:developfrom
ymnliu:develop

Conversation

@ymnliu
Copy link
Copy Markdown
Contributor

@ymnliu ymnliu commented Nov 21, 2020

Add date annotator to the current template for #8 .

@ymnliu
Copy link
Copy Markdown
Contributor Author

ymnliu commented Nov 21, 2020

Hi @tschaffter, can you check on why the check did not pass? It may be related to CI/CD pipelines.

@tschaffter
Copy link
Copy Markdown
Member

There is a lint issue in server/Dockerfile that hadolint started to report recently.

server/Dockerfile:35 DL3042 Avoid use of cache directory with pip. Use `pip install --no-cache-dir <package>`

Here is how to fix server/Dockerfile

  • remove the line ENV PIP_NO_CACHE_DIR=off
  • replace RUN pip install --no-cache-dir -r ${APP_DIR}/requirements.txt by RUN pip install -r ${APP_DIR}/requirements.txt

@tschaffter tschaffter linked an issue Nov 21, 2020 that may be closed by this pull request
@tschaffter tschaffter merged commit 11125fe into nlpsandbox:develop Nov 27, 2020
@github-actions
Copy link
Copy Markdown
Contributor

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Nov 28, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement date annotation

2 participants