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

Add naturallist #110

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

akshaybabloo
Copy link

@akshaybabloo akshaybabloo commented Apr 21, 2023

  • This PR adds naturallist to convert a list of items into a comma separated sentence with and at the last.
  • I have also ignored .idea/* folder

closes #103

@codecov
Copy link

codecov bot commented Apr 21, 2023

Codecov Report

Merging #110 (14d76fb) into main (7e20f6e) will decrease coverage by 0.78%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #110      +/-   ##
==========================================
- Coverage   99.06%   98.28%   -0.78%     
==========================================
  Files           9       11       +2     
  Lines         745      759      +14     
==========================================
+ Hits          738      746       +8     
- Misses          7       13       +6     
Flag Coverage Δ
macos-latest 96.70% <100.00%> (-1.15%) ⬇️
ubuntu-latest 96.70% <100.00%> (-1.15%) ⬇️
windows-latest 95.25% <100.00%> (-0.72%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/humanize/__init__.py 81.81% <100.00%> (-18.19%) ⬇️
src/humanize/lists.py 100.00% <100.00%> (ø)
tests/test_lists.py 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

@akshaybabloo akshaybabloo changed the title Add naturalist Add naturallist Apr 22, 2023
@hugovk hugovk added the changelog: Added For new features label Apr 22, 2023
@akshaybabloo
Copy link
Author

Could you please run the CI again? Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog: Added For new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature: Natural list
2 participants