Skip to content

Add LensSerialNumber#57

Merged
mindeng merged 1 commit into
mindeng:mainfrom
mqudsi:exif-lens-serial-number
May 23, 2026
Merged

Add LensSerialNumber#57
mindeng merged 1 commit into
mindeng:mainfrom
mqudsi:exif-lens-serial-number

Conversation

@mqudsi
Copy link
Copy Markdown
Contributor

@mqudsi mqudsi commented May 23, 2026

Found this while comparing the fields we need for some forensics. Exiftool supports it, fwiw.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.84%. Comparing base (383d0f6) to head (f384199).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #57   +/-   ##
=======================================
  Coverage   89.83%   89.84%           
=======================================
  Files          38       38           
  Lines        8393     8396    +3     
=======================================
+ Hits         7540     7543    +3     
  Misses        853      853           
Flag Coverage Δ
nom-exif 89.84% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mindeng mindeng merged commit f0030f0 into mindeng:main May 23, 2026
4 checks passed
mindeng added a commit that referenced this pull request May 23, 2026
…alNumber EXIF tags

Minor release adding two standard EXIF tag definitions:

- CameraSerialNumber (0xa431) — camera body serial number (#56)
- LensSerialNumber  (0xa435) — lens serial number (#57)

Both additive only, fully backward compatible.
@mindeng
Copy link
Copy Markdown
Owner

mindeng commented May 23, 2026

Thanks for the contribution! Both CameraSerialNumber (#56) and LensSerialNumber (#57) are now included in nom-exif v3.5.0, just published to crates.io. Appreciate the help expanding tag coverage!

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