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

Release 1.1.2 #10

Closed
wants to merge 2 commits into from
Closed

Release 1.1.2 #10

wants to merge 2 commits into from

Conversation

JesusMcCloud
Copy link
Contributor

needs a valid mdoc response

@nodh
Copy link
Contributor

nodh commented Dec 20, 2024

Suggesting to move on, will be fixed with a-sit-plus/vck#185

@JesusMcCloud JesusMcCloud marked this pull request as ready for review December 20, 2024 08:20
@JesusMcCloud
Copy link
Contributor Author

publishing, but keeping this PR open until a fix lands in VC-K

@nodh
Copy link
Contributor

nodh commented Dec 23, 2024

Fix in vc-k is released in 5.2.1

@JesusMcCloud
Copy link
Contributor Author

Tried with latest published VC-K snapshot (which should contain the fix, but it doesn't even compile). So maybe something else is the current latest snapshot.

what do we do? it works in VC-K as regression test. just merge and keep it failing? personally I don't think it is worth messing with this, since the regression tests in VC-K will catch any hiccups anyhow

@nodh
Copy link
Contributor

nodh commented Jan 8, 2025

See #11

@nodh nodh closed this Jan 8, 2025
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