Skip to content

Add WORKSPACE_SERVICE_PREFIX to make user_resource_bundle#4731

Merged
tamirkamara merged 4 commits into
mainfrom
jc-wk/makefile-workspace-service-prefix
Oct 30, 2025
Merged

Add WORKSPACE_SERVICE_PREFIX to make user_resource_bundle#4731
tamirkamara merged 4 commits into
mainfrom
jc-wk/makefile-workspace-service-prefix

Conversation

@JC-wk

@JC-wk JC-wk commented Oct 27, 2025

Copy link
Copy Markdown
Collaborator

Resolves #4721

What is being addressed

make user_resource_bundle doesn't work if your bundle is not named tre-service-{NAME}

How is this addressed

Add WORKSPACE_SERVICE_PREFIX to make user_resource_bundle for use where a workspace-service bundle name does not start with tre-service-

@JC-wk JC-wk changed the title makefile workspace service prefix Add WORKSPACE_SERVICE_PREFIX to make user_resource_bundle Oct 27, 2025
@github-actions

github-actions Bot commented Oct 27, 2025

Copy link
Copy Markdown

Unit Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
0 files    0 ❌

Results for commit 74d46a9.

♻️ This comment has been updated with latest results.

@tamirkamara tamirkamara left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, but I can't be the first approver. And, this PR is in draft.

@JC-wk JC-wk marked this pull request as ready for review October 28, 2025 19:39
@JC-wk JC-wk requested a review from a team as a code owner October 28, 2025 19:39
@JC-wk

JC-wk commented Oct 28, 2025

Copy link
Copy Markdown
Collaborator Author

@tamirkamara thanks, I have set it to ready for review

@marrobi marrobi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Maybe needs to be in the docs/pipelines?

@tamirkamara

Copy link
Copy Markdown
Collaborator

/test-force-approve 74d46a9

@github-actions

Copy link
Copy Markdown

🤖 pr-bot 🤖

✅ Marking tests as complete (for commit 74d46a9)

(in response to this comment from @tamirkamara)

@tamirkamara tamirkamara merged commit 101116c into main Oct 30, 2025
12 checks passed
@tamirkamara tamirkamara deleted the jc-wk/makefile-workspace-service-prefix branch October 30, 2025 11:16
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.

make user_resource_bundle has hardcoded service name but service naming is inconsistent

3 participants