Skip to content
This repository has been archived by the owner on Dec 1, 2021. It is now read-only.

fix some typos #439

Merged
merged 2 commits into from
Sep 19, 2019
Merged

fix some typos #439

merged 2 commits into from
Sep 19, 2019

Conversation

tkng
Copy link
Contributor

@tkng tkng commented Sep 18, 2019

fixed some typos

Motivation and Context

I just wanted to fix some typos

Description

Checked some files by aspell.

How has this been tested?

unit test

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature / Optimization (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@blueoil-butler blueoil-butler bot added the CI: auto-run Run CI automatically label Sep 18, 2019
Copy link
Contributor

@tsawada tsawada left a comment

Choose a reason for hiding this comment

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

wow this is a lot! Nice work :)

Copy link
Member

@iizukak iizukak left a comment

Choose a reason for hiding this comment

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

LGTM except @tsawada san's comment.

@iizukak iizukak requested a review from tsawada September 19, 2019 06:07
@tsawada
Copy link
Contributor

tsawada commented Sep 19, 2019

LGTM, Approval

@iizukak iizukak merged commit e7c6d6a into blue-oil:master Sep 19, 2019
ananno pushed a commit to ananno/blueoil that referenced this pull request Sep 27, 2019
* fix some typos

* fix mis-correction
@iizukak iizukak added this to the v0.12.0 milestone Oct 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CI: auto-run Run CI automatically
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants