[PR #897] [MERGED] Update version script, and unify the linting so that prettier is more consistent #946

Closed
opened 2026-02-16 00:19:37 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/run-llama/llama_cloud_services/pull/897
Author: @adrianlyjak
Created: 9/4/2025
Status: Merged
Merged: 9/4/2025
Merged by: @adrianlyjak

Base: mainHead: adrian/lint-version


📝 Commits (1)

  • baec044 Add version script, and unify the linting so that prettier is more consistent

📊 Changes

11 files changed (+341 additions, -65 deletions)

View changed files

📝 .github/workflows/build_package_ts.yml (+0 -2)
📝 .github/workflows/lint.yml (+12 -1)
.github/workflows/lint_ts.yml (+0 -37)
📝 .github/workflows/release_ts.yml (+0 -2)
📝 .github/workflows/test_ts.yml (+4 -7)
📝 .pre-commit-config.yaml (+6 -4)
package.json (+19 -0)
📝 pnpm-lock.yaml (+245 -0)
📝 scripts/version-bump.py (+53 -11)
📝 ts/llama_cloud_services/package.json (+1 -0)
📝 ts/llama_cloud_services/src/reader.ts (+1 -1)

📄 Description

I got sidetracked here with a few general repo infra changes while working on another change:

  • since there's no changesets support yet, just added a quicky js version thing to the existing script
  • noticed that there were inconsisent prettier formats being enforced. The pre-commit prettier was disagreeing with the actual package.json prettier. This led to a rabbit hole of:
    • running lint-staged for js checks within pre-commit
    • unifying the linter github actions
    • removing some other redundant checks between the many github actions

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/run-llama/llama_cloud_services/pull/897 **Author:** [@adrianlyjak](https://github.com/adrianlyjak) **Created:** 9/4/2025 **Status:** ✅ Merged **Merged:** 9/4/2025 **Merged by:** [@adrianlyjak](https://github.com/adrianlyjak) **Base:** `main` ← **Head:** `adrian/lint-version` --- ### 📝 Commits (1) - [`baec044`](https://github.com/run-llama/llama_cloud_services/commit/baec04451eb11106eb9cf113a87c0312dd700a86) Add version script, and unify the linting so that prettier is more consistent ### 📊 Changes **11 files changed** (+341 additions, -65 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/build_package_ts.yml` (+0 -2) 📝 `.github/workflows/lint.yml` (+12 -1) ➖ `.github/workflows/lint_ts.yml` (+0 -37) 📝 `.github/workflows/release_ts.yml` (+0 -2) 📝 `.github/workflows/test_ts.yml` (+4 -7) 📝 `.pre-commit-config.yaml` (+6 -4) ➕ `package.json` (+19 -0) 📝 `pnpm-lock.yaml` (+245 -0) 📝 `scripts/version-bump.py` (+53 -11) 📝 `ts/llama_cloud_services/package.json` (+1 -0) 📝 `ts/llama_cloud_services/src/reader.ts` (+1 -1) </details> ### 📄 Description I got sidetracked here with a few general repo infra changes while working on another change: - since there's no changesets support yet, just added a quicky js version thing to the existing script - noticed that there were inconsisent prettier formats being enforced. The pre-commit prettier was disagreeing with the actual package.json prettier. This led to a rabbit hole of: - running lint-staged for js checks within pre-commit - unifying the linter github actions - removing some other redundant checks between the many github actions --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-16 00:19:37 -05:00
yindo closed this issue 2026-02-16 00:19:38 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: run-llama/llama_cloud_services#946