Skip to content
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

Make upgrade script executable #7111

Merged
merged 1 commit into from
Jan 17, 2025
Merged

Make upgrade script executable #7111

merged 1 commit into from
Jan 17, 2025

Conversation

ychebotarev
Copy link
Contributor

@ychebotarev ychebotarev commented Jan 17, 2025

What changed?

change the attributes of upgrade.sh

Why?

Making this script executable allows automation. Most likely this is not used, but just in case.

Potential risks

No as I aware

Documentation

N/A

Is hotfix candidate?

Yes

@ychebotarev ychebotarev requested a review from a team as a code owner January 17, 2025 19:22
@alexshtin alexshtin changed the title make upgrade script executable Make upgrade script executable Jan 17, 2025
@ychebotarev ychebotarev merged commit faabd96 into main Jan 17, 2025
50 checks passed
@ychebotarev ychebotarev deleted the y_fix branch January 17, 2025 20:00
stephanos pushed a commit to stephanos/temporal that referenced this pull request Jan 17, 2025
## What changed?
<!-- Describe what has changed in this PR -->
change the attributes of upgrade.sh

## Why?
<!-- Tell your future self why have you made these changes -->
Making this script executable allows automation. Most likely this is not
used, but just in case.

## Potential risks
<!-- Assuming the worst case, what can be broken when deploying this
change to production? -->
No as I aware

## Documentation
<!-- Have you made sure this change doesn't falsify anything currently
stated in `docs/`? If significant
new behavior is added, have you described that in `docs/`? -->
N/A

## Is hotfix candidate?
<!-- Is this PR a hotfix candidate or does it require a notification to
be sent to the broader community? (Yes/No) -->
Yes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants