Skip to content
This repository has been archived by the owner on Jan 29, 2024. It is now read-only.

delete runtime to allow build in both netlify and cloudflare pages #1645

Merged
merged 1 commit into from
Dec 16, 2022

Conversation

angelinekwan
Copy link
Collaborator

@angelinekwan angelinekwan commented Dec 16, 2022

What changed, and why it matters

Currently docs.aiven.io is hosted in Netlify which support the latest Python version 3.8. As we are migrating the deployment platform to Cloudflare Pages, currently the build support version up to Python 3.7 only.

Prior to the switch of deployment platform, build need to support both Netlify and Cloudflare Pages. To avoid conflict of version, we can remove this runtime.txt and set the Python version as environment variable in both Netlify and Cloudflare Pages. Documentation here.

Related to Issue 1528

Preview

Build successfully in both Netlify and Cloudflare Pages.
Netlify https://deploy-preview-1645--developer-aiven-io-preview.netlify.app/
Cloudflare Pages https://delete-runtime-allow-build-n.devportal.pages.dev/

@netlify
Copy link

netlify bot commented Dec 16, 2022

Deploy Preview for developer-aiven-io-preview ready!

Name Link
🔨 Latest commit 682df5b
🔍 Latest deploy log https://app.netlify.com/sites/developer-aiven-io-preview/deploys/639c343e9f8b9e0009cdd64e
😎 Deploy Preview https://deploy-preview-1645--developer-aiven-io-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Copy link
Contributor

@lornajane lornajane left a comment

Choose a reason for hiding this comment

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

approved

@lornajane lornajane merged commit 7c28cd0 into main Dec 16, 2022
@lornajane lornajane deleted the delete-runtime-allow-build-netlify-cloudflare branch December 16, 2022 13:52
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.

2 participants