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

Update formatting for rustfmt 1.4.12-stable #312

Closed
wants to merge 1 commit into from

Conversation

dfreese
Copy link
Contributor

@dfreese dfreese commented May 5, 2020

The handling of rustfmt for the varint! macro changes between rustfmt
1.4.11-stable and rustfmt 1.4.12-stable. The new formatting is
acceptable to both versions.

The handling of rustfmt for the varint! macro changes between rustfmt
1.4.11-stable and rustfmt 1.4.12-stable.  The new formatting is
acceptable to both versions.
dfreese added a commit to dfreese/prost that referenced this pull request May 5, 2020
I've noticed a number of github actions reporting failing or cancelled
on the cache step.  This removes them, for now, to provide a better
signal for the project, until why the cache action fails can be
determined.

Builds on PR tokio-rs#312.
@danburkert
Copy link
Collaborator

Thanks for the PR. I ran rustfmt on another PR before seeing this, so the changes have already landed.

@danburkert danburkert closed this May 10, 2020
danburkert added a commit that referenced this pull request May 10, 2020
* Update formatting for rustfmt 1.4.12-stable

The handling of rustfmt for the varint! macro changes between rustfmt
1.4.11-stable and rustfmt 1.4.12-stable.  The new formatting is
acceptable to both versions.

* Remove cache action step that is causing problems for CI

I've noticed a number of github actions reporting failing or cancelled
on the cache step.  This removes them, for now, to provide a better
signal for the project, until why the cache action fails can be
determined.

Builds on PR #312.

* remove all caching

Co-authored-by: David Freese <[email protected]>
@dfreese dfreese deleted the rustfmt branch May 17, 2020 19:02
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