When we introduced native methods, AmlContext lost its Send + Sync, as methods weren't constrained to be thread-safe. This was fixed in #91 by requiring them to be, but it's unfortunate that this wasn't picked up before then (and so we have broken releases, which is fine per our semver commitments, but not great).
A test that makes sure that AmlContext is Send + Sync should be pretty easy and will trip CI if we make a change that breaks this in the future.
When we introduced native methods,
AmlContextlost itsSend + Sync, as methods weren't constrained to be thread-safe. This was fixed in #91 by requiring them to be, but it's unfortunate that this wasn't picked up before then (and so we have broken releases, which is fine per our semver commitments, but not great).A test that makes sure that
AmlContextisSend + Syncshould be pretty easy and will trip CI if we make a change that breaks this in the future.