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

remove ollama summarizer and frontmatter_checking things for the new module #642

Conversation

kjaymiller
Copy link
Contributor

Issue Link 🔗

Resolves #638

Type of Change

  • Bug fix 🐞
  • New feature/page
  • Documentation update
  • Other

Description 📋

  • What: Provide an overview of the issue this PR addresses. Explain the context and background information.

Removes the frontmatter checking maintained here in favor of frontmatter-check pre-commit recipe

  • Why: Describe why the changes are being made. Highlight key updates, new features, or bug fixes.

Reduces the complexity of the site

  • How: Explain how these changes will affect the project or end-users.

No change to end users

Checklist ✅

  • Followed the Code of Conduct and Contribution Guide
  • Ran pre-commit run --all
  • All tests pass locally
  • Added tests (if applicable)
  • Documentation updated (if applicable)

Additional Notes & Screenshots

Add any additional notes or comments that might be helpful for the reviewers.

@kjaymiller kjaymiller linked an issue Jan 29, 2025 that may be closed by this pull request
2 tasks
@kjaymiller
Copy link
Contributor Author

Sorry for the changes around the workflows... Things were breaking but it was because of Python Version issues.

@kjaymiller kjaymiller merged commit 7e003ea into gh-pages Jan 29, 2025
6 checks passed
@kjaymiller kjaymiller deleted the 638-remove-frontmatter-checking-code-to-use-new-frontmatter-check-pre-commit branch January 29, 2025 16:47
@kjaymiller kjaymiller mentioned this pull request Jan 29, 2025
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.

remove frontmatter checking code to use new frontmatter-check pre-commit.
2 participants