8000 Update docs with new crevice version by sampullman · Pull Request #1274 · ggez/ggez · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Update docs with new crevice version #1274

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

Merged
merged 3 commits into from
Apr 3, 2024

Conversation

sampullman
Copy link

Related #1084

@nobbele I've been getting back into rust/ggez dev and looking through a few issues for low hanging fruit.

I could be mistaken, but don't think this is possible to solve without significant changes to crevice, since they hard code crate references in the proc macros.

This PR just updates the docs with the latest crevice version and adds a few notes in the examples. The issue can probably be closed.

@sampullman
Copy link
Author

skeptic on the beta channel is failing to compile doc tests/examples. It seems it's an issue with detecting dependencies, but I'm not too familiar with skeptic.

This fixes doctests by setting skeptic to the branch in:
budziq/rust-skeptic#142

It's a temporary fix because skeptic is no longer maintained.
@sampullman sampullman force-pushed the 1084-reexport-crevice branch from bff12d5 to b4eb1bb Compare March 31, 2024 09:35
@@ -71,7 +71,7 @@ keyframe = "1"
keyframe_derive = "1"
num-derive = "0.4"

skeptic = "0.13"
skeptic = { git = "https://github.com/Erk-/rust-skeptic.git", branch = "fix-rust-1-77" }
8000
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@a1phyr This is a temp fix so we can keep doctests running in CI until I can find an alternative for skeptic.

I cherry-picked your other CI fixes here to test everything together.

@sampullman sampullman merged commit fdd6f75 into ggez:devel Apr 3, 2024
@sampullman sampullman mentioned this pull request Apr 3, 2024
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
0