Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Improve event json metadata #710

Merged
merged 7 commits into from
Sep 11, 2018
Merged

Improve event json metadata #710

merged 7 commits into from
Sep 11, 2018

Conversation

bkchr
Copy link
Member

@bkchr bkchr commented Sep 11, 2018

No description provided.

pub struct Module<T: Trait> for enum Call where origin: T::Origin {}
}

impl_event!(
Copy link
Member

Choose a reason for hiding this comment

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

prefer to keep naming consistently decl_ for this stuff

}
impl_event!(
pub enum Event<T> with RawEvent<AccountId, AccountIndex, Balance>
where <T as system::Trait>::AccountId, <T as Trait>::AccountIndex, <T as Trait>::Balance {
Copy link
Member

Choose a reason for hiding this comment

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

to avoid the problem of the indentation change not reflecting the block start, we move the trailing curly to the previous indent level and give it its own line in these circumstances., so this would become:

pub enum Event<T> with RawEvent<AccountId, AccountIndex, Balance>
	where <T as system::Trait>::AccountId, <T as Trait>::AccountIndex, <T as Trait>::Balance
{
	/// A new account was created.

if the where clause becomes too big then this works:

pub enum Event<T> with RawEvent<AccountId, AccountIndex, Balance> where
	<T as system::Trait>::AccountId,
	<T as Trait>::AccountIndex,
	<T as Trait>::Balance
{
	/// A new account was created.

Copy link
Member

@gavofyork gavofyork left a comment

Choose a reason for hiding this comment

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

Some style nits that can be fixed, but otherwise looks great.

@gavofyork gavofyork merged commit 1ea42fc into master Sep 11, 2018
@gavofyork gavofyork deleted the improve_event_json_metadata branch September 11, 2018 12:05
liuchengxu added a commit to chainx-org/substrate that referenced this pull request Aug 23, 2021
* Add EvilValidatorsPerSession and fix other nits

* Split out slash.rs in staking
helin6 pushed a commit to boolnetwork/substrate that referenced this pull request Jul 25, 2023
Bumps [Swatinem/rust-cache](https://github.com/Swatinem/rust-cache) from 2.0.1 to 2.2.0.
- [Release notes](https://github.com/Swatinem/rust-cache/releases)
- [Changelog](https://github.com/Swatinem/rust-cache/blob/master/CHANGELOG.md)
- [Commits](Swatinem/rust-cache@22c9328...359a70e)

---
updated-dependencies:
- dependency-name: Swatinem/rust-cache
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants