Skip to content

docs: fix stale fork note and erroneous deploy steps in README#144

Merged
ewega merged 3 commits intomainfrom
copilot/update-readme-remove-copilot-blurb
Mar 16, 2026
Merged

docs: fix stale fork note and erroneous deploy steps in README#144
ewega merged 3 commits intomainfrom
copilot/update-readme-remove-copilot-blurb

Conversation

Copy link
Contributor

Copilot AI commented Mar 14, 2026

The README contained inaccurate information in two areas that needed correction.

Changes

  • Removed stale fork note: Dropped the [!NOTE] callout stating the Copilot metrics plugin was only available in the DevExpGBB/incubator-devlake fork — that upstream PR has since merged into the official Apache images.
  • Removed erroneous manual Docker steps: The Quick Start and Step-by-step sections incorrectly instructed users to run cd devlake && docker compose up -d manually. This is handled automatically by gh devlake deploy local, which starts the containers and waits until DevLake is healthy — no manual intervention needed.
Original prompt

The Readme mentions the GitHub Copilot plugin is only in a specific fork. The pr has now been merged please update accordingly and remove that blurb


📍 Connect Copilot coding agent with Jira, Azure Boards or Linear to delegate work to Copilot in one click without leaving your project management tool.

Co-authored-by: ewega <26189114+ewega@users.noreply.github.com>
Copilot AI changed the title [WIP] Update README to remove Copilot plugin fork mention docs: remove stale fork note for GitHub Copilot plugin Mar 14, 2026
Copilot AI requested a review from ewega March 14, 2026 18:25
@ewega
Copy link
Contributor

ewega commented Mar 14, 2026

@copilot I'm also noticing the deploy steps in the Readme might be erroneous. Why are we telling users to do docker compose up ? That's being handled by the gh-devlake

Co-authored-by: ewega <26189114+ewega@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Mar 14, 2026

@copilot I'm also noticing the deploy steps in the Readme might be erroneous. Why are we telling users to do docker compose up ? That's being handled by the gh-devlake

Good catch — gh devlake deploy local calls docker compose up -d internally and polls until DevLake is healthy, so users never need to run it manually. Removed the erroneous cd devlake && docker compose up -d lines from both the Quick Start and Step-by-step sections, and updated the description to reflect that the command handles the full lifecycle (commit 4f624b7).

Copilot AI changed the title docs: remove stale fork note for GitHub Copilot plugin docs: fix stale fork note and erroneous deploy steps in README Mar 14, 2026
@ewega ewega marked this pull request as ready for review March 16, 2026 17:35
Copilot AI review requested due to automatic review settings March 16, 2026 17:35
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the project README to remove outdated guidance about Copilot plugin availability and to correct local deployment instructions so they match how gh devlake deploy local is intended to be used.

Changes:

  • Removed a stale note about the Copilot metrics plugin only being available in a fork.
  • Removed manual docker compose up -d steps from Quick Start and step-by-step deploy instructions.
  • Updated the Step 1 deploy description to state that deploy local starts the stack and waits for health.

```

Downloads Docker Compose files, generates secrets, and prepares the stack. Give it ~2 minutes after `docker compose up`. See [docs/deploy.md](docs/deploy.md) for flags and details.
Downloads Docker Compose files, generates secrets, starts the stack, and waits until DevLake is healthy. See [docs/deploy.md](docs/deploy.md) for flags and details.
> **GitHub Copilot plugin:** The Copilot metrics plugin is currently available in the [DevExpGBB/incubator-devlake](https://github.com/DevExpGBB/incubator-devlake) fork while the upstream PR ([apache/incubator-devlake#8728](https://github.com/apache/incubator-devlake/pull/8728)) is under review. The `deploy local` and `deploy azure` commands handle custom images automatically — no manual image builds needed. Once the PR merges, the official Apache images will include it.

<!-- SCREENSHOT: Grafana DORA dashboard + Copilot Adoption dashboard side-by-side — the "payoff" -->
<!-- SCREENSHOT:Grafana DORA dashboard + Copilot Adoption dashboard side-by-side — the "payoff" -->
@ewega ewega merged commit ca2b3ac into main Mar 16, 2026
11 checks passed
@ewega ewega deleted the copilot/update-readme-remove-copilot-blurb branch March 16, 2026 17:46
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.

3 participants