Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix typo in README.md: depenedency to dependency #70

Merged
merged 3 commits into from
Jun 15, 2022

Conversation

moreaupascal56
Copy link
Contributor

Pull Request
Are you a current Fivetran customer?
Pascal Moreau Data Engineer Yousign
What change(s) does this PR introduce?
None, just a really small typo fix in readme (I couldn't resist fixing it ahah :) )

Did you update the CHANGELOG?
no
Does this PR introduce a breaking change?

  • Yes (please provide breaking change details below.)
  • [ x] No (please provide an explanation as to how the change is non-breaking below.)

Provide an emoji that best describes your current mood

💃

Feedback

We are so excited you decided to contribute to the Fivetran community dbt package! We continue to work to improve the packages and would greatly appreciate your feedback on our existing dbt packages or what you'd like to see next.

Copy link
Contributor

@fivetran-joemarkiewicz fivetran-joemarkiewicz left a comment

Choose a reason for hiding this comment

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

@moreaupascal56 thanks so much opening this PR to correct the spelling mistake in the README! Eagle eyes like yours are exactly what we need to keep these READMEs in check.

I just made one small commit to your patch to add your contribution to the CHANGELOG. I will merge this into main and it will be included in the next release! Thanks again 🏅

@fivetran-joemarkiewicz fivetran-joemarkiewicz merged commit 5bc141c into fivetran:main Jun 15, 2022
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.

2 participants