search

elliotchance / gedcom

👪 A Go library and CLI tools for encoding, decoding, traversing, merging, comparing, querying and publishing GEDCOM files.
MIT License
97 stars 22 forks source link

Publish: Fix surname index #292

Closed elliotchance closed 4 years ago

elliotchance commented 4 years ago

The bug was caused by taking the first letter of the name, which may not be the first letter of the surname.

This change is Reviewable

codecov-io commented 4 years ago

Codecov Report

Merging #292 into master will increase coverage by 0.03%. The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #292      +/-   ##
==========================================
+ Coverage   71.52%   71.56%   +0.03%     
==========================================
  Files         201      201              
  Lines        6094     6094              
==========================================
+ Hits         4359     4361       +2     
+ Misses       1611     1610       -1     
+ Partials      124      123       -1
Impacted Files Coverage Δ
html/individual_index_header.go 47.05% <100%> (ø) :arrow_up:
individual_node.go 95.85% <50%> (+0.48%) :arrow_up:

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 e0f9651...e373a63. Read the comment docs.