Skip to content

disable cc-service-bot's management of the Semaphore pipeline contents #298

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 25, 2025

Conversation

cchristous
Copy link
Member

@cchristous cchristous commented Apr 25, 2025

Please prefix all TypeScript pull-requests with [Typescript]

What

This changes the service.yml so that cc-service-bot will no longer try to manage the contents of .semaphore/semaphore.yml.

Checklist

  • Contains customer facing changes? Including API/behavior changes
  • Did you add sufficient unit test and/or integration test coverage for this PR?
    • If not, please explain why it is not required

References

JIRA:

Test & Review

N/A - can only be tested on the default branch

Open questions / Follow-ups

@Copilot Copilot AI review requested due to automatic review settings April 25, 2025 22:01
@cchristous cchristous requested review from a team as code owners April 25, 2025 22:01
Copy link

@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 pull request updates the service configuration to disable cc-service-bot’s management of the Semaphore pipeline by setting the new key "pipeline_enable" to false.

  • Disable automatic management of the Semaphore pipeline
  • Update service.yml configuration accordingly
Comments suppressed due to low confidence (1)

service.yml:13

  • Consider adding tests to verify that setting 'pipeline_enable' to false prevents cc-service-bot from managing the Semaphore pipeline as intended.
pipeline_enable: false

Copy link
Member

@kcorman0 kcorman0 left a comment

Choose a reason for hiding this comment

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

thanks was just looking at this for another repo 👍

@cchristous cchristous merged commit 6449094 into master Apr 25, 2025
1 of 2 checks passed
@cchristous cchristous deleted the DP-16417 branch April 25, 2025 22:15
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