Companion for #8615: enrich metadata with type information

[ascjones]

Companion for paritytech/substrate#8615.

Cumulus companion: paritytech/cumulus#564

todo:

• a scale-info release (Tracking issue for 1.0 scale-info#89),
• merge Derive scale_info::TypeInfo for types used in polkadot grandpa-bridge-gadget#218

First look at json generated metadata: https://gist.github.com/ascjones/726caaf29eff76ca72f142f419e9c6a9

[ascjones]

bot merge

[ghost]

Error: When trying to meet the "Project Owners" approval requirements: this pull request does not belong to a project defined in Process.json.

Approval by "Project Owners" is only attempted if other means defined in the criteria for merge are not satisfied first.

The following errors might have affected the outcome of this attempt:

• 'Batch: Availability and Validity' does not match any projects in polkadot's Process.json
• 'Batch: Codebase Restructure' does not match any projects in polkadot's Process.json

[ordian]

bot merge

[ghost]

Waiting for commit status.

[ordian]

(fmt and spellcheck are ignored by bot)

[ordian]

the check-transaction-versions if failing though:

2021-09-15 12:50:29        API/INIT: Error: FATAL: Unable to initialize the API: Cannot convert metadata from version 14 to 13
    at EventEmitter.value (/usr/local/lib/node_modules/@polkadot/metadata-cmp/node_modules/@polkadot/api/base/Init.cjs:82:25)
    at processTicksAndRejections (node:internal/process/task_queues:94:5)
    at runNextTicks (node:internal/process/task_queues:63:3)
    at processImmediate (node:internal/timers:437:9)

[jacogr]

Indeed. 2 issues -

1. The API PR has not been merged/released as of yet (so the comparator doesn't understand v14), see Parsing for metadata v14 polkadot-js/api#3759
2. The metadata compare has not been revamped for v14 (there is some work left since the type compares now need to be done differently), see Adapt metadata compare for v14 polkadot-js/tools#267

1 above will happen first, 2 above needs to wait until after polkadot-js/apps#5927 (no dependency between 2 and that, however I can only focus/do one at a time)

[bkchr]

@jacogr So, I assume we should disable the check until it works again?

[jacogr]

Hopefully sorted by Monday, aka need to get all released by then. But yes, would disable it at present.

[ascjones]

It seems the removed check is lingering.

[ordian]

@TriplEight ^^^

[shawntabrizi]

bot merge

[ghost]

Trying merge.

[ghost]

Merge aborted: Head SHA changed from c86a008 to ff5e6b7