Skip to content

chore: update DEPLOY.md — deploy-api-services now in dpla/api#26

Merged
DominicBM merged 1 commit into
mainfrom
chore/deploy-script
Apr 15, 2026
Merged

chore: update DEPLOY.md — deploy-api-services now in dpla/api#26
DominicBM merged 1 commit into
mainfrom
chore/deploy-script

Conversation

@DominicBM
Copy link
Copy Markdown
Contributor

@DominicBM DominicBM commented Apr 14, 2026

Summary

  • Updates DEPLOY.md reference for deploy-api-services: the script now lives at bin/deploy-api-services in the dpla/api repository (see dpla/api#new PR)
  • Replaces "ask in #tech" with an actionable copy command

Test plan

  • Verify the copy command works: cp bin/deploy-api-services ~/bin/ && chmod +x ~/bin/deploy-api-services (run from dpla/api repo)

🤖 Generated with Claude Code

Summary

Updates DEPLOY.md to reflect that the deploy-api-services script is now located in the dpla/api repository at bin/deploy-api-services. The documentation now provides explicit instructions to copy and make the script executable (cp bin/deploy-api-services ~/bin/ && chmod +x ~/bin/deploy-api-services), replacing the previous guidance to "ask in #tech" if the script is not available.

Changes: Documentation-only update to DEPLOY.md (+1/-1 lines)

The combined deploy script is now versioned at bin/deploy-api-services
in the dpla/api repo. Updated the reference so operators know where to
find it rather than "ask in #tech".

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 14, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 84f1f46c-5b99-4e76-a6d7-ecb084c44a63

📥 Commits

Reviewing files that changed from the base of the PR and between e512d0a and f916fdf.

📒 Files selected for processing (1)
  • DEPLOY.md

Walkthrough

Documentation update to DEPLOY.md clarifying the location of the deploy-api-services script. Instructions now specify retrieving it from the dpla/api repository and provide explicit steps for copying and making it executable in the local ~/bin/ directory.

Changes

Cohort / File(s) Summary
Deployment Documentation
DEPLOY.md
Updated guidance for obtaining the deploy-api-services script: now specifies the dpla/api repository location with explicit copy and chmod instructions instead of generic reference to alternate sources.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: updating DEPLOY.md to reflect that the deploy-api-services script is now located in the dpla/api repository.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/deploy-script

Comment @coderabbitai help to get the list of available commands and usage tips.

@DominicBM
Copy link
Copy Markdown
Contributor Author

@coderabbitai full review

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 14, 2026

✅ Actions performed

Full review triggered.

@DominicBM
Copy link
Copy Markdown
Contributor Author

@coderabbitai review

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 15, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@DominicBM
Copy link
Copy Markdown
Contributor Author

@coderabbitai full review

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 15, 2026

✅ Actions performed

Full review triggered.

@DominicBM
Copy link
Copy Markdown
Contributor Author

@coderabbitai full review

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 15, 2026

✅ Actions performed

Full review triggered.

@DominicBM
Copy link
Copy Markdown
Contributor Author

@coderabbitai full review

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 15, 2026

✅ Actions performed

Full review triggered.

@DominicBM DominicBM merged commit 908feb9 into main Apr 15, 2026
6 checks passed
@DominicBM DominicBM deleted the chore/deploy-script branch April 15, 2026 20:06
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.

1 participant