Skip to content

Conversation

@dvdplm
Copy link
Contributor

@dvdplm dvdplm commented Sep 23, 2020

Update dependencies and sort out incompatibilities.

}

#[cfg(test)]
#[cfg(feature = "integration-tests")]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

All tests in this module are integration tests so cargo check --tests will warn about unused items without this.

Copy link
Contributor

Choose a reason for hiding this comment

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

Means the individual test ones are redundant now, but anyway I'll fix that in #165

@ascjones ascjones mentioned this pull request Sep 23, 2020
Sort out the RefCount type (ty @dvc94ch)
Random tweaks to make test-node more similar to the vanilla node-template
@dvdplm dvdplm mentioned this pull request Sep 24, 2020
@dvdplm dvdplm marked this pull request as ready for review September 24, 2020 10:17

/// Type used to encode the number of references an account has.
pub type RefCount = u8;
pub type RefCount = u32;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@dvc94ch ty! Was scratching my head on this one. :)

Copy link
Contributor

@ascjones ascjones left a comment

Choose a reason for hiding this comment

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

LGTM (test failure just the usual pure virtual method called)

}

#[cfg(test)]
#[cfg(feature = "integration-tests")]
Copy link
Contributor

Choose a reason for hiding this comment

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

Means the individual test ones are redundant now, but anyway I'll fix that in #165

@ascjones ascjones merged commit 8e2a4f0 into master Sep 24, 2020
@ascjones ascjones deleted the dp-upgrade-substrate-to-2.0 branch September 24, 2020 11:57
dvdplm added a commit that referenced this pull request Sep 24, 2020
* master:
  Upgrade to substrate 2.0.0 (#173)
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