Skip to content

Add Github action for building binaries - #9

Merged
AleksandrParamonoff merged 1 commit into
mainfrom
add-github-action-for-building-binaries
Dec 16, 2025
Merged

AleksandrParamonoff merged 1 commit into
mainfrom
add-github-action-for-building-binaries

Conversation

@AleksandrParamonoff

Copy link
Copy Markdown
Collaborator

No description provided.

@AleksandrParamonoff
AleksandrParamonoff merged commit ecd0b61 into main Dec 16, 2025
7 checks passed

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

This PR adds artifact upload functionality to the existing GitHub Actions build workflow. The change enables automatic uploading of built binaries as workflow artifacts, but only for commits pushed to the main branch.

Key Changes:

  • Added artifact upload step that uploads the bin/ directory contents with 30-day retention
  • Upload is conditional and only executes for pushes to the main branch

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

if: github.ref == 'refs/heads/main'
uses: actions/upload-artifact@v4
with:
name: healenium-mcp-server-binaries

Copilot AI Dec 16, 2025

Copy link

Choose a reason for hiding this comment

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

The artifact name uses "binaries" (plural), but only a single Linux binary is built on ubuntu-latest. Consider renaming to "healenium-mcp-server-linux" or "healenium-mcp-server-binary" to more accurately reflect what's being uploaded.

Suggested change
name: healenium-mcp-server-binaries
name: healenium-mcp-server-linux

Copilot uses AI. Check for mistakes.
- name: Run build
run: task app:build

# Upload built binaries as artifacts (only for main branch)

Copilot AI Dec 16, 2025

Copy link

Choose a reason for hiding this comment

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

The comment says "binaries" (plural), but since this workflow runs on ubuntu-latest, only a single Linux binary will be built and uploaded. Consider updating the comment to say "Upload built binary" (singular) or "Upload Linux binary" to be more accurate.

Suggested change
# Upload built binaries as artifacts (only for main branch)
# Upload built binary as artifact (only for main branch)

Copilot uses AI. Check for mistakes.
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