Skip to content

fix(iso4217): remove BGN as Bulgaria adopted EUR on 2026-01-01#1584

Open
64johnlee wants to merge 1 commit into
go-playground:masterfrom
64johnlee:fix-remove-bgn-iso4217-1517
Open

fix(iso4217): remove BGN as Bulgaria adopted EUR on 2026-01-01#1584
64johnlee wants to merge 1 commit into
go-playground:masterfrom
64johnlee:fix-remove-bgn-iso4217-1517

Conversation

@64johnlee

Copy link
Copy Markdown

Summary

Bulgaria replaced the Bulgarian lev (BGN, ISO 4217 numeric 975) with the Euro on January 1, 2026. This PR removes BGN from the iso4217 alpha map and 975 from the iso4217_numeric map so the validator reflects the current ISO 4217 standard.

Closes #1517

Changes

  • currency_codes.go — removed "BGN": {} from iso4217 alpha map and 975: {} from iso4217_numeric map
  • validator_test.go — added {"BGN", false} to TestIsIso4217Validation and {975, false} to TestIsIso4217NumericValidation to assert BGN/975 are no longer accepted

Test plan

  • go build ./... passes
  • go test ./... passes (all packages)
  • New negative test cases confirm BGN (alpha) and 975 (numeric) are rejected

Bulgaria replaced the Bulgarian lev (BGN, numeric 975) with the Euro
on January 1, 2026. Remove BGN from the iso4217 alpha map and 975 from
the iso4217 numeric map so the validator reflects the current standard.
Add negative test cases to confirm BGN/975 are no longer accepted.

Closes go-playground#1517

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@64johnlee
64johnlee requested a review from a team as a code owner June 9, 2026 15:58

@nodivbyzero nodivbyzero 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

@64johnlee

Copy link
Copy Markdown
Author

Just a gentle ping — this PR has an approval from @nodivbyzero and all CI checks are green across Go 1.25/1.26 on Ubuntu, macOS, and Windows. Happy to rebase if needed, but it looks ready to merge whenever you get a chance. Thanks!

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.

Removal of BGN ccy from the list

2 participants