Skip to content

Conversation

@hfhoffman1144
Copy link
Contributor

Where to put new files:

  • New files should go into a top-level subfolder, named after the article slug. For example: my-awesome-article

How to merge your changes:

  1. Make sure the CI code style tests all pass (+ run the automatic code formatter if necessary).
  2. Find an RP Team member on Slack and ask them to review & approve your PR.
  3. Once the PR has one positive ("approved") review, GitHub lets you merge the PR.
  4. 🎉

hfhoffman1144 and others added 4 commits January 29, 2025 08:16
Removed linters and formatters from pyproject.toml
Created a requirements.txt file
Updated the README file
Ran isort to fix import spacing and order
Copy link
Contributor

@martin-martin martin-martin left a comment

Choose a reason for hiding this comment

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

Hi @hfhoffman1144 , I've applied a few changes directly to your PR, they're just general-level, but LMK if you want to reverse anything.

I have one request about adding a quick usage example.

Also, there may or may not be some changes to the code here after addressing my comments in the tutorial PR.

@martin-martin
Copy link
Contributor

A classic descriptive commit that resolves a unclear situation 🤣

image

Thanks for applying the updates here as well :)

@brendaweles brendaweles merged commit 3372cd7 into master Mar 3, 2025
1 check passed
@brendaweles brendaweles deleted the langgraph branch March 3, 2025 18:44
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.

4 participants