Skip to content

chore(dev): add Dockerfile.jailoc for in-container development#77

Merged
thejoeejoee merged 6 commits into
seznam:mainfrom
speedo17:chore/add-dev-dockerfile
Apr 16, 2026
Merged

chore(dev): add Dockerfile.jailoc for in-container development#77
thejoeejoee merged 6 commits into
seznam:mainfrom
speedo17:chore/add-dev-dockerfile

Conversation

@speedo17

Copy link
Copy Markdown
Contributor

Adds a workspace overlay Dockerfile for developing jailoc inside a jailoc container, with Go toolchain, gopls, and golangci-lint matching CI. Documents the setup in both AGENTS.md and docs/development.md.

Copilot AI review requested due to automatic review settings April 16, 2026 13:00

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

Adds an optional “workspace overlay” Dockerfile to enable developing jailoc from inside a jailoc container, aligning the in-container toolchain (Go + gopls + golangci-lint) with what CI expects, and documents how to enable it.

Changes:

  • Add Dockerfile.jailoc overlay image that installs Go, gopls, and golangci-lint on top of the resolved base image.
  • Document the in-container development workflow in docs/development.md and AGENTS.md.
  • Ignore .envrc files in git.

Reviewed changes

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

File Description
docs/development.md Adds contributor docs for running jailoc development inside a jailoc container via Dockerfile.jailoc.
Dockerfile.jailoc New overlay Dockerfile that installs Go, gopls, and golangci-lint into the container.
AGENTS.md Adds “Development environment” guidance for using the overlay Dockerfile.
.gitignore Ignores .envrc.

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

Comment thread docs/development.md
Comment thread docs/development.md Outdated
Comment thread Dockerfile.jailoc Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

@thejoeejoee thejoeejoee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

prosim

  • presunout do dev/Dockerfile[.jailoc]
  • dockerfile = pro vyvoj linkovat jako url z githubu, e.g. neco jako dockerfile = "https://raw.githubusercontent.com/seznam.cz/jailoc/main/Dockerfile.jailoc", snad mam tu url dobre

@speedo17

Copy link
Copy Markdown
Contributor Author
  • dockerfile presunuty,
  • v doc linkovany pres url

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

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.


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

Comment thread dev/Dockerfile.jailoc
Comment thread dev/Dockerfile.jailoc
@thejoeejoee thejoeejoee self-requested a review April 16, 2026 18:54
@thejoeejoee thejoeejoee merged commit 9bda33e into seznam:main Apr 16, 2026
7 checks passed
@thejoeejoee thejoeejoee added the docker Container, compose, DinD, image resolution label Apr 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docker Container, compose, DinD, image resolution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants