Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix: add libssl-dev, perl, make to musl cross Dockerfiles for openssl-sys build#160

Draft
Copilot wants to merge 17 commits into
masterfrom
copilot/fix-remaining-failing-os-sequences
Draft

fix: add libssl-dev, perl, make to musl cross Dockerfiles for openssl-sys build#160
Copilot wants to merge 17 commits into
masterfrom
copilot/fix-remaining-failing-os-sequences

Conversation

Copy link
Copy Markdown

Copilot AI commented May 3, 2026

  • Restore libssl-dev, perl, make to both musl cross Dockerfiles
  • Revert src/lib/sub_commands/query.rs to its original pre-change state
  • Rebase: squash 5 messy commits into single clean commit on top of master

Copilot AI and others added 11 commits May 3, 2026 00:01
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 3, 2026

Thanks for your contribution!

This PR doesn't have a linked issue. All PRs must reference an existing issue.

Please:

  1. Open an issue describing the bug/feature (if one doesn't exist)
  2. Add Fixes #<number> or Closes #<number> to this PR description

See CONTRIBUTING.md for details.

@RandyMcMillan RandyMcMillan force-pushed the copilot/fix-remaining-failing-os-sequences branch from cdd3279 to 8f9f6e4 Compare May 3, 2026 13:46
Copy link
Copy Markdown

@RandyMcMillan RandyMcMillan left a comment

Choose a reason for hiding this comment

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

revert this change

Copy link
Copy Markdown
Author

Copilot AI commented May 3, 2026

revert this change

Done — libssl-dev, perl, and make removed from both musl cross Dockerfiles in 1f7d078.

Copilot AI changed the title fix: add libssl-dev, perl, make to musl cross Dockerfiles for openssl-sys build revert: remove libssl-dev, perl, make from musl cross Dockerfiles May 3, 2026
Copilot AI requested a review from RandyMcMillan May 3, 2026 13:55
Copy link
Copy Markdown

@RandyMcMillan RandyMcMillan left a comment

Choose a reason for hiding this comment

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

you just broke this pr with this commit

Copy link
Copy Markdown
Author

Copilot AI commented May 3, 2026

you just broke this pr with this commit

Restored libssl-dev, perl, and make to both musl cross Dockerfiles in 62ae456.

Copilot AI changed the title revert: remove libssl-dev, perl, make from musl cross Dockerfiles fix: add libssl-dev, perl, make to musl cross Dockerfiles for openssl-sys build May 3, 2026
Copilot AI requested a review from RandyMcMillan May 3, 2026 14:04
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 3, 2026

Hey! Your PR title revert: remove libssl-dev, perl, make from musl cross Dockerfiles doesn't follow conventional commit format.

Please update it to start with one of:

  • feat: or feat(scope): new feature
  • fix: or fix(scope): bug fix
  • docs: or docs(scope): documentation changes
  • chore: or chore(scope): maintenance tasks
  • refactor: or refactor(scope): code refactoring
  • test: or test(scope): adding or updating tests

Where scope is the package name (e.g., app, desktop, opencode).

See CONTRIBUTING.md for details.

Copy link
Copy Markdown

@RandyMcMillan RandyMcMillan left a comment

Choose a reason for hiding this comment

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

this file change src/lib/sub_commands/query.rs is breaking

Copy link
Copy Markdown
Author

Copilot AI commented May 3, 2026

this file change src/lib/sub_commands/query.rs is breaking

Reverted src/lib/sub_commands/query.rs to its original pre-change state in 7e7508c.

Copilot AI requested a review from RandyMcMillan May 3, 2026 14:06
Copy link
Copy Markdown

@RandyMcMillan RandyMcMillan left a comment

Choose a reason for hiding this comment

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

rebase

Copy link
Copy Markdown
Author

Copilot AI commented May 3, 2026

rebase

The branch is already based on the current tip of master (63f58cf). No rebase is needed — the branch is up to date with master.

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