-
Notifications
You must be signed in to change notification settings - Fork 6
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
Update to new stable #![doc = include_str!]
#4
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Closes #2 (probably) |
I'd like to fix #1 before merging this just to have some checks in place, but otherwise this looks good to go! |
Cool! I’m happy to submit a workflow file if you like? |
Sure! If not, I'll get to it this weekend. |
I might only have time on Monday but will try find a gap 👍 |
Hi @hdevalence I added a github actions workflow that should mirror what travis ci was doing, example run here: https://github.com/delta1/subtle-ng/actions/runs/1203724120 |
Awesome, thanks! |
Hi folks! We're trying to move to stable rust so I thought I'd contribute this change upstream :)
Removes
feature(external_doc)
in favor of#![doc = include_str!]
introduced in stable Rust 1.54.Fixed up the tests imports and bumped the version - I understand you may not want these changes but thought I'd offer them for now!
Cheers