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

rename polkadot-collator to polkadot-parachain #1241

Merged
merged 11 commits into from
May 13, 2022

Conversation

gilescope
Copy link
Contributor

Not every node has to be a collator.

Fixes #979

Not every node has to be a collator.
@gilescope gilescope requested review from a team and chevdor as code owners May 6, 2022 12:41
@gilescope gilescope requested review from joepetrowski and bkchr May 6, 2022 12:57
@paritytech-ci paritytech-ci requested a review from a team May 6, 2022 14:20
Copy link
Contributor

@joepetrowski joepetrowski left a comment

Choose a reason for hiding this comment

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

Approving the name change, but someone more familiar with CI/versioning should review that everything works as expected.

@chevdor
Copy link
Contributor

chevdor commented May 6, 2022

Yes please hold off a bit on merging, I would like to check carefully and we need to update some other repos.

Copy link
Contributor

@chevdor chevdor left a comment

Choose a reason for hiding this comment

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

A few points to fix IMO but thanks for the big renaming!

.github/workflows/extrinsic-ordering-check-from-bin.yml Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
polkadot-parachains/Cargo.toml Outdated Show resolved Hide resolved
@gilescope gilescope requested a review from skunert May 10, 2022 10:25
gilescope added 2 commits May 10, 2022 16:16
but allows for patch releases by having a 0 at the end.
(`-patch1` patch naming schemes were discussed but they
were judged to cause downstream packagers pain.)
@gilescope gilescope requested a review from chevdor May 10, 2022 15:27
Copy link
Contributor

@chevdor chevdor left a comment

Choose a reason for hiding this comment

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

I am not convinced about this 0.9.190 scheme but I prefer it to the -1 options, everyone seems to agree about the need to follow the prolkadot version and I don't want to slow this down.

Copy link
Contributor

@hbulgarini hbulgarini left a comment

Choose a reason for hiding this comment

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

When the node is started without --collator option the node still prints the Polkadot Collator log:

./polkadot-collator --tmp --port 40337 --ws-port 9948 -- --execution wasm --chain polkadot-local-raw.json  --port 30337
2022-05-12 13:45:26 Polkadot collator
2022-05-12 13:45:26 ✌️  version 5.4.0-4bbedb30b
...
2022-05-12 13:45:26 Is collating: no

Copy link
Contributor

@hbulgarini hbulgarini left a comment

Choose a reason for hiding this comment

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

LGTM

@gilescope
Copy link
Contributor Author

bot merge

@paritytech-processbot paritytech-processbot bot merged commit 8d44f9b into master May 13, 2022
@paritytech-processbot paritytech-processbot bot deleted the giles-rename-collator branch May 13, 2022 13:02
gilescope added a commit that referenced this pull request May 13, 2022
* rename polkadot-collator to polkadot-parachain

Not every node has to be a collator.

* Update README.md

Co-authored-by: Chevdor <[email protected]>

* rename docker file

* Update .github/workflows/extrinsic-ordering-check-from-bin.yml

Co-authored-by: Chevdor <[email protected]>

* Versioning scheme that tracks polkadot relay chain

but allows for patch releases by having a 0 at the end.
(`-patch1` patch naming schemes were discussed but they
were judged to cause downstream packagers pain.)

* update name

* update lock file

Co-authored-by: Chevdor <[email protected]>
jiguantong added a commit to darwinia-network/darwinia-parachain that referenced this pull request Jun 29, 2022
jiguantong added a commit to darwinia-network/darwinia-parachain that referenced this pull request Jul 9, 2022
� Conflicts:
�	Cargo.lock
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.

Rename polkadot-collator bin
8 participants