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

Ensure that README code example is up to date #1017

Closed
wants to merge 1 commit into from
Closed

Ensure that README code example is up to date #1017

wants to merge 1 commit into from

Conversation

GuillaumeGomez
Copy link
Contributor

This provide a simple way to check that your README example is up-to-date.

@@ -306,6 +306,13 @@ extern crate unicode_xid;
#[cfg(feature = "printing")]
extern crate quote;

#[cfg(doctest)]
Copy link
Contributor

Choose a reason for hiding this comment

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

Note that this increases MSRV to 1.40. BurntSushi/termcolor#35

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Outch, good point!

Copy link
Owner

@dtolnay dtolnay left a comment

Choose a reason for hiding this comment

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

I would prefer not to make this change. You've sent equivalent PRs to some of my other projects in the past, which were similarly rejected. Could you please avoid sending any more of these to all projects under my account?

@dtolnay dtolnay closed this Apr 10, 2021
@GuillaumeGomez GuillaumeGomez deleted the readme-example branch April 10, 2021 18:06
@GuillaumeGomez
Copy link
Contributor Author

Sorry, was using syn in one of my project and realized that, completely forgot I already sent the same change proposal to other project of yours. Will try to not do it again.

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.

3 participants