Skip to content

feat: add exiftool to runtime packages - #115

Open
jaysomani wants to merge 1 commit into
appwrite:mainfrom
jaysomani:feat/exiftool
Open

jaysomani wants to merge 1 commit into
appwrite:mainfrom
jaysomani:feat/exiftool

Conversation

@jaysomani

Copy link
Copy Markdown

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

@jaysomani
jaysomani marked this pull request as ready for review October 8, 2026 11:26
@hansi-codes

hansi-codes Bot commented Oct 8, 2026

Copy link
Copy Markdown

🟢 Tier S · Ready to merge

The package addition and matching availability/version test are consistent with the image build and test setup.

The final Docker image now installs ExifTool. The container structure test checks that the exiftool command is available and reports a 13.x version.

Verdict New comments Fixed Still open
💬 Commented 0 0 0

Note

@jaysomani does not have write access to this repository, so Hansi does not approve automatically. A maintainer can review and approve.

📂 Walkthrough · 2
File Change
Dockerfile Installs the ExifTool package in the final image.
tests.yaml Adds a command test for the ExifTool executable and its version.

Reviewed c2d110b · Details · Comment @hansi-codes review to re-run, or mention @hansi-codes with a question.

@hansi-codes hansi-codes Bot 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.

🟢 Tier S · Looks good to merge. Summary

@jaysomani jaysomani changed the title inital work feat: add exiftool to runtime packages Oct 8, 2026

@Meldiron Meldiron 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.

lgtm, opportunity for php extension too, maybe

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