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.

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

Bump infrahub-sdk to v1.2.0 #33

Closed
wants to merge 8 commits into from
Closed

Conversation

wvandeun
Copy link
Contributor

No description provided.

@wvandeun wvandeun self-assigned this Dec 20, 2024
@wvandeun wvandeun requested a review from a team December 20, 2024 08:21
@wvandeun wvandeun force-pushed the wvd-20241220-bump-sdk-1.2.0 branch from 3b7303a to 7adbc21 Compare December 23, 2024 08:45
@wvandeun wvandeun force-pushed the wvd-20241220-bump-sdk-1.2.0 branch from 7adbc21 to 42d72f9 Compare December 23, 2024 08:49
@FragmentedPacket FragmentedPacket dismissed their stale review December 23, 2024 14:39

Potential issue with nornir

@FragmentedPacket
Copy link

I added some commits to continue on error for Python 3.12 and Nornir.

I believe we should not run into that issue once Nornir 3.5.0 is released and pending this PR: nornir-automation/nornir#981

We could also just not test 3.12, continue-on-error on the specific step (shows all tests passing so no feedback on failure), or just bypass the failed check for 3.12.

@FragmentedPacket
Copy link

Closing in favor of #34 due to newer SDK release.

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