Skip to content

Fix the rake workflow on Ubuntu and Windows runners#12

Open
suleman-uzair wants to merge 4 commits into
mainfrom
fix/ci-green
Open

Fix the rake workflow on Ubuntu and Windows runners#12
suleman-uzair wants to merge 4 commits into
mainfrom
fix/ci-green

Conversation

@suleman-uzair

Copy link
Copy Markdown

This PR fixes both platform failures in the rake workflow. Ubuntu jobs ran all 882 examples green and then aborted in lint:registrybin/lint-no-raw-mdc has no shebang line, so runners execute it with /bin/sh (dash), which rejects set -o pipefail. Windows jobs never get that far: actions/checkout fails with invalid path because one tracked filename starts with \x05, a character Windows forbids in paths.

Changes

  • Add #!/usr/bin/env bash to bin/lint-no-raw-mdc — the script uses bash-only syntax (pipefail, arrays), and macOS runners passed only because /bin/sh is bash there.
  • Rename reference-docs/parcelmaker/Parcel Maker5.2.1/\x05SummaryInformation to SummaryInformation — the \x05 prefix is Microsoft's reserved OLE stream name, written to disk verbatim when the ParcelMaker MSI was unpacked; git records a 100% rename and the original .msi stays tracked alongside, so no reference content is lost.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 3 changed files in this pull request and generated no new comments.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 4 changed files in this pull request and generated 1 comment.

Comment thread spec/model/entity_store_spec.rb Outdated
@suleman-uzair
suleman-uzair marked this pull request as ready for review July 17, 2026 14:08
@suleman-uzair
suleman-uzair requested a review from Copilot July 17, 2026 14:08

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 4 changed files in this pull request and generated no new comments.

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