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

Fixed Typo #1022

Merged
merged 1 commit into from
Aug 4, 2024
Merged

Fixed Typo #1022

merged 1 commit into from
Aug 4, 2024

Conversation

eltociear
Copy link
Contributor

Pull Request Checklist

lenght -> length

Below is a simple checklist but please do not hesitate to ask for assistance!

  • Fork, clone, and checkout the newest version of the code
  • Create a new branch
  • Make necessary code changes
  • Install black (i.e., python -m pip install black or conda install -c conda-forge black)
  • Install flake8 (i.e., python -m pip install flake8 or conda install -c conda-forge flake8)
  • Install pytest-cov (i.e., python -m pip install pytest-cov or conda install -c conda-forge pytest-cov)
  • Run black . in the root stumpy directory
  • Run flake8 . in the root stumpy directory
  • Run ./setup.sh && ./test.sh in the root stumpy directory
  • Reference a Github issue (and create one if one doesn't already exist)

lenght -> length
@eltociear eltociear requested a review from seanlaw as a code owner August 4, 2024 11:28
@seanlaw seanlaw changed the title chore: update core.py Fixed Typo Aug 4, 2024
@seanlaw
Copy link
Contributor

seanlaw commented Aug 4, 2024

@eltociear Thank you for fixing this typo!

@seanlaw seanlaw merged commit 79096a8 into TDAmeritrade:main Aug 4, 2024
33 checks passed
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