Skip to content

doc: update readme for recent changes - #2059

Merged
farmdawgnation merged 2 commits into
mainfrom
farmdawgnation/conch
Sep 21, 2026
Merged

farmdawgnation merged 2 commits into
mainfrom
farmdawgnation/conch

Conversation

@farmdawgnation

Copy link
Copy Markdown
Member

This updates the lift readme for accuracy with recent changes to the codebase.

Copilot AI lite review requested due to automatic review settings September 6, 2026 15:16

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The changes are documentation-only and appear internally consistent with the current project/module structure and referenced tooling.

Pull request overview

This PR updates the repository README to reflect the current recommended development workflow and module/dependency structure for Lift, aligning instructions with recent changes in the codebase and tooling ecosystem.

Changes:

  • Updates the “Running the Server” instructions to use sbt-war tasks (warStart, warStartPackage, warStop) and explains their behavior.
  • Replaces the legacy xsbt-web-plugin guidance with sbt-war plugin setup and updated reference link.
  • Corrects dependency/module relationships and sbt subproject command examples (e.g., lift-webkit project naming and transitive dependency notes).
File summaries
File Description
README.md Refreshes server-run instructions, sbt plugin guidance, and dependency/project naming details to match current Lift setup.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@farmdawgnation
farmdawgnation merged commit 72d30bf into main Sep 21, 2026
11 of 12 checks passed
@farmdawgnation
farmdawgnation deleted the farmdawgnation/conch branch September 21, 2026 00:56
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.

2 participants