Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

matifali
Copy link
Member

@matifali matifali commented Aug 23, 2025

Description

Fixes the source URL in README.md

Type of Change

  • New module
  • Bug fix
  • Feature/enhancement
  • Documentation
  • Other

Module Information

Path: registry/coder-labs/modules/sourcegraph-amp
New version: v1.0.1
Breaking change: [ ] Yes [x] No

Testing & Validation

  • Tests pass (bun test)
  • Code formatted (bun run fmt)
  • Changes tested locally

Related Issues

@Copilot Copilot AI review requested due to automatic review settings August 23, 2025 18:48
Copy link
Contributor

@Copilot 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

This PR fixes the Sourcegraph AMP module URL by updating the module source path to use hyphens instead of underscores, ensuring consistency with the module's actual registry path. It also bumps the version from v1.0.0 to v1.0.1.

  • Updated module source URL from sourcegraph_amp to sourcegraph-amp in documentation examples
  • Bumped module version from 1.0.0 to 1.0.1
  • Updated module name references to use consistent hyphen format

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@matifali matifali requested review from bpmct and 35C4n0r August 23, 2025 18:48
@matifali matifali added the version:patch Add to PRs requiring a patch version upgrade label Aug 23, 2025
35C4n0r
35C4n0r previously approved these changes Aug 23, 2025
Copy link
Collaborator

@35C4n0r 35C4n0r left a comment

Choose a reason for hiding this comment

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

LGTM!

@matifali matifali requested a review from 35C4n0r August 23, 2025 19:15
@matifali matifali enabled auto-merge (squash) August 23, 2025 19:15
@matifali matifali merged commit ac48f0d into main Aug 23, 2025
4 checks passed
@matifali matifali deleted the atif/fix-source-graph-amp-source-url branch August 23, 2025 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
version:patch Add to PRs requiring a patch version upgrade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants