Skip to content

Fix shebang and quotes in mirror scripts documentation#558

Merged
utkarsh2102 merged 2 commits into
ubuntu:mainfrom
zyga:patch-1
Jul 9, 2026
Merged

Fix shebang and quotes in mirror scripts documentation#558
utkarsh2102 merged 2 commits into
ubuntu:mainfrom
zyga:patch-1

Conversation

@zyga

@zyga zyga commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

Description

Fix shebang and add quotes to make scripts pass shellcheck


Checklist

  • I have read and followed the Ubuntu Project contributing guide
  • My pull request is linked to an existing issue (if applicable)
  • I have tested my changes, and they work as expected

Copilot AI review requested due to automatic review settings April 17, 2026 08:18
@github-actions github-actions Bot added the Release team For the attention of the Release team label Apr 17, 2026

Copilot AI 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.

Pull request overview

This PR updates the Release Team documentation for mirror sync scripts to correct the dash shebang and improve quoting in the trace-file path so the example scripts are closer to shellcheck-compliant.

Changes:

  • Fix the dash shebang in both example scripts (#/bin/dash#!/bin/dash).
  • Add quoting around $(hostname -f) in the trace file redirection paths.

Comment thread docs/release-team/mirror-scripts.md Outdated
Comment thread docs/release-team/mirror-scripts.md Outdated
@utkarsh2102

Copy link
Copy Markdown
Member

@copilot resolve the merge conflicts in this pull request

@utkarsh2102 utkarsh2102 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.

thank you!

@utkarsh2102 utkarsh2102 merged commit bbea64d into ubuntu:main Jul 9, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Release team For the attention of the Release team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants