Skip to content
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

ExifTagDescriptionAttribute doesnt need fields #978

Merged
merged 2 commits into from
Sep 17, 2019
Merged

ExifTagDescriptionAttribute doesnt need fields #978

merged 2 commits into from
Sep 17, 2019

Conversation

SimonCropp
Copy link
Contributor

since the reflection is done on the constructor arguments

Note that i am not familiar with the AOT approach in use here. so please verify my assumption

since the reflection is done on the constructor arguments
@codecov
Copy link

codecov bot commented Aug 15, 2019

Codecov Report

Merging #978 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #978      +/-   ##
==========================================
+ Coverage    89.7%   89.71%   +<.01%     
==========================================
  Files        1097     1097              
  Lines       48556    48554       -2     
  Branches     3422     3422              
==========================================
  Hits        43558    43558              
+ Misses       4295     4293       -2     
  Partials      703      703
Impacted Files Coverage Δ
...aData/Profiles/Exif/ExifTagDescriptionAttribute.cs 63.63% <ø> (+9.79%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5643486...3f43783. Read the comment docs.

@codecov
Copy link

codecov bot commented Aug 15, 2019

Codecov Report

Merging #978 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #978      +/-   ##
==========================================
+ Coverage   89.74%   89.75%   +<.01%     
==========================================
  Files        1098     1098              
  Lines       48690    48688       -2     
  Branches     3433     3433              
==========================================
  Hits        43698    43698              
+ Misses       4288     4286       -2     
  Partials      704      704
Impacted Files Coverage Δ
...aData/Profiles/Exif/ExifTagDescriptionAttribute.cs 63.63% <ø> (+9.79%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0cda174...af28443. Read the comment docs.

@JimBobSquarePants JimBobSquarePants merged commit 644fe22 into SixLabors:master Sep 17, 2019
@JimBobSquarePants JimBobSquarePants added this to the 1.0.0-rc1 milestone Apr 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants