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

docs(js): update vercel and langchain examples add readme for openinference-core #1083

Merged
merged 5 commits into from
Oct 25, 2024

Conversation

Parker-Stafford
Copy link
Contributor

  • update vercel example to remove experimental telemetry
  • update langchain example to use ^0.3.0
  • add readme and docs for openinference core trace config and context attributes

@Parker-Stafford Parker-Stafford requested a review from a team as a code owner October 25, 2024 05:51
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Oct 25, 2024
@Parker-Stafford Parker-Stafford changed the title docs: update vercel and langchain examples add readme for openinference-core docs(js): update vercel and langchain examples add readme for openinference-core Oct 25, 2024
@@ -5,8 +5,8 @@ FROM base AS deps
# Check https://github.com/nodejs/docker-node/tree/b4117f9333da4138b03a546ec926ef50a31506c3#nodealpine to understand why libc6-compat might be needed.
RUN apk add --no-cache libc6-compat
WORKDIR /app
COPY package*.json ./
RUN npm ci
Copy link
Contributor Author

Choose a reason for hiding this comment

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

we removed pacage locks from examples

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 25, 2024
@Parker-Stafford Parker-Stafford merged commit 5c0f1e5 into main Oct 25, 2024
3 checks passed
@Parker-Stafford Parker-Stafford deleted the parker/fix-docs-add-core-docs branch October 25, 2024 15:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants