Skip to content

Relocate config-server to src/#13

Open
aramprice wants to merge 1 commit into
mainfrom
remove-gopath-ism
Open

Relocate config-server to src/#13
aramprice wants to merge 1 commit into
mainfrom
remove-gopath-ism

Conversation

@aramprice

@aramprice aramprice commented Jul 2, 2026

Copy link
Copy Markdown
Member

In modern golang there's no need to use
github.com/cloudfoundry/config-server/ directory prefixing.

Merge after #15

In modern golang there's no need to use
`github.com/cloudfoundry/config-server/` directory prefixing.
@aramprice aramprice force-pushed the remove-gopath-ism branch from dc598af to 63601b7 Compare July 8, 2026 00:23
ystros
ystros previously approved these changes Jul 8, 2026

@ystros ystros left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Assuming those integration asset files weren't used, seem fine.

@github-project-automation github-project-automation Bot moved this from Inbox to Pending Merge | Prioritized in Foundational Infrastructure Working Group Jul 8, 2026
@aramprice

Copy link
Copy Markdown
Member Author

Assuming those integration asset files weren't used, seem fine.

turns out it is needed but somehow I missed that.. probably git switch .. fighting with itself 🙃

@aramprice aramprice requested a review from ystros July 8, 2026 16:41
@aramprice aramprice dismissed ystros’s stale review July 8, 2026 16:41

issues found 😞

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Pending Merge | Prioritized

Development

Successfully merging this pull request may close these issues.

2 participants