Skip to content

#2086 Zeropad unique suffix IDs#2087

Merged
whitwham merged 2 commits into
samtools:developfrom
cjw85:patch-1
Jul 30, 2024
Merged

#2086 Zeropad unique suffix IDs#2087
whitwham merged 2 commits into
samtools:developfrom
cjw85:patch-1

Conversation

@cjw85

@cjw85 cjw85 commented Jul 23, 2024

Copy link
Copy Markdown
Contributor

See #2086

Formatting of suffix used the code "%0lX", the 0 says to pad with '0' (not ' '), but no field width is specified resulting in no zero-padding

cjw85 added 2 commits July 23, 2024 17:56
Formatting of suffix used the code `"%0lX"`, the `0` says to pad with `'0'` (not `' '`), but no field width is specified resulting in no zero-padding
@cjw85

cjw85 commented Jul 24, 2024

Copy link
Copy Markdown
Contributor Author

I've updated the expected test results, I must confess I don't know why some of the @PG lines are deleted when I run make test. All other changes show the expected zero-padded suffixes at a fixed width of 8.

@whitwham whitwham merged commit 190a62a into samtools:develop Jul 30, 2024
@whitwham

Copy link
Copy Markdown
Member

Merged, thank you.

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