Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add pipeline to build cookbooks #292

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Add pipeline to build cookbooks #292

wants to merge 1 commit into from

Conversation

invader89
Copy link
Collaborator

@invader89 invader89 commented Dec 9, 2024

This PR updates the Cookbooks MDX files generated from the latest notebooks.

Changes

  • Added a new file, `authors.yaml, which contains the names and email addresses of the authors.
  • Added a new file, `registry.yaml, which contains the titles, paths, slugs, blurbs, and tags of the notebooks.
  • Added a new file, `build_cookbooks.py, which contains the code for building the Cookbooks MDX files.</co: 1>
  • Added a new file, `templates/cookbook.md, which contains the template for the Cookbooks MDX files.
  • Added a new file, `build-cookbooks.yml, which contains the code for building the Cookbooks MDX files.</co: 4>
  • Added a new file, `poetry.lock, which contains the dependencies for the project.
  • Updated the pyproject.toml file to include the new dependencies.

Copy link

github-actions bot commented Dec 9, 2024

Comment on lines +33 to +36
- name: Install Poetry
shell: bash
run: |
pip install poetry # Using pip to install Poetry
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
- name: Install Poetry
shell: bash
run: |
pip install poetry # Using pip to install Poetry
- name: Bootstrap poetry
uses: snok/install-poetry@v1
with:
version: 1.5.1
virtualenvs-in-project: false

Copy link
Collaborator

Choose a reason for hiding this comment

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

let's move to scripts folder

Comment on lines +4 to +9
workflow_dispatch:
inputs:
branch:
description: 'Branch to run the workflow on'
required: true
default: 'main'
Copy link
Collaborator

Choose a reason for hiding this comment

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

I will follow up, hopefully we can move cookbooks to this repo and then trigger on pr

@billytrend-cohere
Copy link
Collaborator

let's add mypy build too which runs on every pr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants