Skip to content

Fix typo.#2059

Closed
rolfschr wants to merge 1 commit intoapache:masterfrom
rolfschr:master
Closed

Fix typo.#2059
rolfschr wants to merge 1 commit intoapache:masterfrom
rolfschr:master

Conversation

@rolfschr
Copy link
Copy Markdown

@rolfschr rolfschr commented Feb 9, 2017

Dear Airflow Maintainers,

Please accept this PR that fixes a typo.

Let me know if I need to create an JIRA issue for that. I could not find "typo" as Types of Contribution in the guidelines for contributing.

@codecov-io
Copy link
Copy Markdown

codecov-io commented Feb 9, 2017

Codecov Report

Merging #2059 into master will not change coverage.

@@          Coverage Diff           @@
##           master   #2059   +/-   ##
======================================
  Coverage    67.6%   67.6%           
======================================
  Files         139     139           
  Lines       10578   10578           
======================================
  Hits         7151    7151           
  Misses       3427    3427
Impacted Files Coverage Δ
airflow/models.py 86.16% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 61fb6a5...3a7c695. Read the comment docs.

@r39132
Copy link
Copy Markdown
Contributor

r39132 commented Feb 9, 2017

Thx for the contribution.. I'm sure you may find other such documentation fixes for your first commit. Please get familiar with our Contributor guidelines (e.g. open a jira, preface commit title with jira issue id) and add a few more doc updates (typo fixes) here. For example, I recall seeing quite a few typos and grammatical issues in UPDATING.md. You may want to fix those if they persist.

https://cwiki.apache.org/confluence/display/AIRFLOW/Contributors%27+Guide

@rolfschr
Copy link
Copy Markdown
Author

Hi,
you can close this PR. I will not open a jira issue for this nor search for other typos. Too much overhead for everbody.

@jlowin
Copy link
Copy Markdown
Member

jlowin commented Feb 11, 2017

@rolfschr only you can close your own PR.

@rolfschr rolfschr closed this Feb 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants