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

some spelling fixes #4088

Merged
merged 5 commits into from
Oct 18, 2021
Merged

some spelling fixes #4088

merged 5 commits into from
Oct 18, 2021

Conversation

ordian
Copy link
Member

@ordian ordian commented Oct 15, 2021

@ordian ordian added A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit. labels Oct 15, 2021
@ordian ordian requested a review from drahnr October 15, 2021 10:50
@@ -62,7 +62,7 @@ impl CollationGenerationSubsystem {
/// Conceptually, this is very simple: it just loops forever.
///
/// - On incoming overseer messages, it starts or stops jobs as appropriate.
/// - On other incoming messages, if they can be converted into Job::ToJob and
/// - On other incoming messages, if they can be converted into `Job::ToJob` and
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

@@ -95,7 +95,7 @@ pub fn channel<T>(
#[allow(missing_docs)]
#[derive(thiserror::Error, Debug)]
pub enum Error {
#[error("Oneshot was cancelled.")]
#[error("Oneshot was canceled.")]
Copy link
Contributor

Choose a reason for hiding this comment

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

@@ -708,7 +708,7 @@ impl<T: Config> Pallet<T> {
///
/// This functions modifies the state when failing. It is expected to be called in inherent,
/// and to fail the extrinsic on error. As invalid inherents are not allowed, the dirty state
/// is not commited.
/// is not committed.
Copy link
Contributor

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

@drahnr drahnr left a comment

Choose a reason for hiding this comment

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

LGTM, added a link about cancelled apparently being OK as well

@drahnr
Copy link
Contributor

drahnr commented Oct 18, 2021

bot merge

@paritytech-processbot
Copy link

Error: You're not authorized to push to this branch. Visit https://docs.github.com/articles/about-protected-branches/ for more information.

@drahnr drahnr merged commit 5f48c2a into master Oct 18, 2021
@drahnr drahnr deleted the ao-some-spelling-fixes branch October 18, 2021 09:59
ordian added a commit that referenced this pull request Oct 27, 2021
* master: (73 commits)
  Fix XCM Teleport Benchmark (#4146)
  Allow Queries and Subscriptions (#4150)
  Fix weights on hard-coded XCM fragments (#4144)
  Bump spec versions (#4142)
  Bump libc from 0.2.104 to 0.2.105 (#4141)
  Enable bags-list pallet in polkadot (#4080)
  Move artifacts states into memory in PVF validation host (#3907)
  Introduce new Runtime API endpoint for fetching the validation data (#3728)
  export xcm_pallet config (#4116)
  Apply cargo fmt (#4122)
  remove logging from the check-dependent-* job (#4120)
  Remove stale migrations (#4107)
  Bump structopt from 0.3.23 to 0.3.25 (#4098)
  remove hardcoded pipeline scripts tag (#4109)
  Bump libc from 0.2.103 to 0.2.104 (#4099)
  some spelling fixes (#4088)
  polkadot: remove call filters on registrar pallets (#4093)
  Fix typos in docs (#4092)
  Rename Statemint where appropriate (#4087)
  bump substrate (#4091)
  ...
emostov pushed a commit that referenced this pull request Nov 1, 2021
* quotes around Job::ToJob
* add new words to dictionary
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants