Skip to content

[FA-32] Implement Dockerfile #16

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

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Conversation

scott-clare1
Copy link
Contributor

@scott-clare1 scott-clare1 commented Apr 4, 2025

This PR adds a Dockerfile for the server to make running the server more consistent.

What

A Dockerfile to run the server.

An entry point for consistently running the server, i.e., it is now runnable as a command line script, e.g., uv run agent-uno

Why

To remove any issues different devices or missing installations and to make it more of a production solution.

How

With a Dockerfile.

Extra

I had to refactor the project so that agent_uno now sits under src this is needed for identifying the project as a command line script.

If new dependencies are introduced to the project, please list them here:

  • click a command line tool.

Checklist

Please ensure you have done the following:

  • I have updated the documentation if required.
  • I have added tests which cover my changes.

Type of change

Make sure to update label on right hand panel.

MacOS tests

To trigger the CI to run on a macOS backed workflow, add the macos-ci-test label to the pull request (PR).

Our advice is to only run this workflow when testing the compatability between operating systems for a change that you've made, e.g., adding a new dependency to the virtual environment.

Note: This can take up to 5 minutes to run. This workflow costs x10 more than a Linux-based workflow, use at discretion.

@scott-clare1 scott-clare1 self-assigned this Apr 4, 2025
@scott-clare1 scott-clare1 added the enhancement New feature or request label Apr 4, 2025
@scott-clare1 scott-clare1 marked this pull request as ready for review April 4, 2025 15:27
Copy link
Member

@Christopher-Norman Christopher-Norman left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants