Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Bug]: cannot import name 'properties' from 'keyring.util' #1111

Closed
CodyCBakerPhD opened this issue Sep 1, 2022 · 3 comments · Fixed by #1112
Closed

[Bug]: cannot import name 'properties' from 'keyring.util' #1111

CodyCBakerPhD opened this issue Sep 1, 2022 · 3 comments · Fixed by #1112
Labels
released

Comments

@CodyCBakerPhD
Copy link
Contributor

Just FYI

Ran into this earlier today: https://github.com/catalystneuro/neuroconv/runs/8136194375?check_suite_focus=true

Already been raised upstream on jaraco/keyring#593

For now I'm pinning to <=23.8.2 downstream, IDK what y'all would prefer to do for now here though.
@yarikoptic
Copy link
Member

Thanks for the alert! Isn't there some != 23.8.2 syntax ? I would assume it would get fixed.. will send a PR shortly.

yarikoptic added a commit that referenced this issue Sep 1, 2022
@yarikoptic
BF(workaround): blacklist 23.8.2 of keyring since introduced regression
1356736 
Closes #1111

Ref upstream: jaraco/keyring#593
@yarikoptic yarikoptic mentioned this issue Sep 1, 2022
Merged
@CodyCBakerPhD CodyCBakerPhD mentioned this issue Sep 1, 2022
Merged
yarikoptic added a commit that referenced this issue Sep 1, 2022
@yarikoptic
BF(workaround): blacklist 23.9.0 of keyring since introduced regression
bf77fc9 
Closes #1111

Ref upstream: jaraco/keyring#593
@github-actions
Copy link

github-actions bot commented Sep 1, 2022

🚀 Issue was released in 0.46.2 🚀

@github-actions
Copy link

github-actions bot commented Sep 1, 2022

🚀 Issue was released in 0.46.2 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
released
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

BF(workaround): blacklist 23.9.0 of keyring since introduced regression dandi/dandi-cli
2 participants
@yarikoptic @CodyCBakerPhD