Skip to content
This repository has been archived by the owner on Mar 11, 2022. It is now read-only.

Fix deployment issues #11

Merged
merged 2 commits into from
Aug 30, 2021
Merged

Fix deployment issues #11

merged 2 commits into from
Aug 30, 2021

Conversation

petterhj
Copy link
Contributor

Should fix compatibility issues for werkzeug 2.x.x: https://stackoverflow.com/a/67554060 and allow deployment for non-linux machines using serverless / dockerizePip: non-linux: lambci/docker-lambda#343

@petterhj petterhj requested a review from a team August 24, 2021 11:23
uses: actions/setup-python@v2
with:
python-version: 3.9
python-version: 3.8
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Funker ikke werkzeug 1.0 med Python 3.9?

Copy link
Contributor Author

@petterhj petterhj Aug 25, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Begynte bare å lure på om det muligens var litt prematurt med 3.9 (ikke werkzeug-relatert); jeg sliter i alle fall med å deploye lokalt-fra på en ikke-linux-maskin (om det er fortsatt noe vi bør kunne gjøre):

docker: Error response from daemon: manifest for lambci/lambda:build-python3.9 not found: 
  manifest unknown: manifest unknown.

ref. lenken over. Får i alle fall ikke testet ting så lett kanskje? :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Der var den samme feilen som jeg så ved deploy ja 😅

Copy link
Contributor

@fredriv fredriv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@petterhj petterhj merged commit 38c7772 into main Aug 30, 2021
@petterhj petterhj deleted the pin-werkzeug-dep branch August 30, 2021 11:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants