Skip to content
This repository was archived by the owner on Jul 3, 2023. It is now read-only.

fix typos in docs and comments #80

Merged
merged 1 commit into from
Feb 20, 2022
Merged

fix typos in docs and comments #80

merged 1 commit into from
Feb 20, 2022

Conversation

jameslamb
Copy link
Contributor

I've been trying to get more familiar with hamilton, and looking for small ways to contribute as I do. Tonight, I tried searching for typos using aspell and found a few. This PR proposes fixing them.

Additions

N/A

Removals

N/A

Changes

  • fixes miscellaneous typos

Testing

N/A

Screenshots

N/A

Notes

N/A

Todos

N/A

Checklist

  • PR has an informative and human-readable title
  • Changes are limited to a single goal (no scope creep)
  • Code can be automatically merged (no conflicts)
  • Code follows the standards laid out in the TODO link to standards
  • Passes all existing automated tests
  • Any change in functionality is tested
  • New functions are documented (with a description, list of inputs, and expected output)
  • Placeholder code is flagged / future todos are captured in comments
  • Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
  • Reviewers requested with the Reviewers tool ➡️

Testing checklist

Python

  • python 3.6
  • python 3.7

@elijahbenizzy
Copy link
Collaborator

@jameslamb thank you! Appreciate the attention to detail :) FYI -- have you joined the discord? https://discord.gg/wCqxqBqn73. Still working to foster a community (we've got some good contributors here but not much action there yet) but its a quick way to ask questions/have community input.

@elijahbenizzy elijahbenizzy self-requested a review February 19, 2022 19:54
@jameslamb
Copy link
Contributor Author

have you joined the discord?

I have! @skrawcz invited me a few days ago 😁

@skrawcz skrawcz merged commit 88f8687 into stitchfix:main Feb 20, 2022
@jameslamb jameslamb deleted the fix/typos branch February 20, 2022 13:21
gitbook-com bot pushed a commit that referenced this pull request Feb 22, 2022
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.

3 participants