Change non-string attribute key from warning to error #1125

Activity

1. sanketverma1704

   added

   good-first-issueGood place to get started as a new contributor.Good place to get started as a new contributor.

   on Oct 11, 2022
2. I-am-Emmanuel commented on Oct 12, 2022

   

   I-am-Emmanuel

   on Oct 12, 2022

   More actions

   It will be my pleasure to work on this issue.
3. I-am-Emmanuel commented on Oct 13, 2022

   

   I-am-Emmanuel

   on Oct 13, 2022

   More actions

   I recently completed this project. Waiting to be merged
4. sanketverma1704 commented on Oct 13, 2022

   

   sanketverma1704

   on Oct 13, 2022

   Last edited by sanketverma1704

   Member

   More actions

   Hi @I-am-Emmanuel. Thanks for working on this.
   We'll review your PR #1183.
5. joshmoore

   removed

   good-first-issueGood place to get started as a new contributor.Good place to get started as a new contributor.

   on Oct 13, 2022
6. sanketverma1704

   added

   maintenanceWork needed by a maintainerWork needed by a maintainer

   on Oct 13, 2022
7. sanketverma1704 commented on Oct 13, 2022

   

   sanketverma1704

   on Oct 13, 2022

   Last edited by sanketverma1704

   Member

   More actions

   Hi @I-am-Emmanuel.
   After going through the issue, I realised this issue is a breaking change and needs to be handled by the maintainers for the Zarr python only.
   I think I added the good-first-issue label accidentally, and I sincerely apologise for that. 🙏🏻

   Can you please work on another issue? I'm happy to help you find one if you can't find one.
8. I-am-Emmanuel commented on Oct 13, 2022

   

   I-am-Emmanuel

   on Oct 13, 2022

   More actions

   Alright

   ❤️React with ❤️1Reacted by Sanket Verma
9. dstansby

   added

   enhancementNew features or improvementsNew features or improvements

   and removed

   maintenanceWork needed by a maintainerWork needed by a maintainer

   on Dec 28, 2024