Skip to content

Add os.listdir #216

New issue

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

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

Already on GitHub? # to your account

Merged
merged 3 commits into from
Jan 17, 2023
Merged

Add os.listdir #216

merged 3 commits into from
Jan 17, 2023

Conversation

reyoung
Copy link
Contributor

@reyoung reyoung commented Jan 16, 2023

No description provided.

@reyoung
Copy link
Contributor Author

reyoung commented Jan 16, 2023

@sbinet @glaukiol1

Please take a review, thanks

@codecov
Copy link

codecov bot commented Jan 16, 2023

Codecov Report

Base: 74.38% // Head: 74.38% // Decreases project coverage by -0.00% ⚠️

Coverage data is based on head (41b8ddf) compared to base (0cc4032).
Patch coverage: 74.19% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #216      +/-   ##
==========================================
- Coverage   74.38%   74.38%   -0.01%     
==========================================
  Files          76       76              
  Lines       12625    12656      +31     
==========================================
+ Hits         9391     9414      +23     
- Misses       2562     2567       +5     
- Partials      672      675       +3     
Impacted Files Coverage Δ
stdlib/os/os.go 67.72% <74.19%> (+0.70%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Member

@sbinet sbinet left a comment

Choose a reason for hiding this comment

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

thanks for the PR.

I have a couple of comments.
see below.

@reyoung
Copy link
Contributor Author

reyoung commented Jan 16, 2023

Perhaps py.ParseTupleAndKeywords needs more comments. It is hard to understand how to set the format parameter.

:-)

@reyoung
Copy link
Contributor Author

reyoung commented Jan 16, 2023

Oh, I see..

py.ParseTupleAndKeywords is just like CPython API ParseTupleAndKeywords.

Perhaps need a comment to show that.

Copy link
Member

@sbinet sbinet left a comment

Choose a reason for hiding this comment

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

LGTM.

could you please send another PR against go-python/license, adding yourself to the AUTHORS and/or CONTRIBUTORS files ?
thanks.

@reyoung
Copy link
Contributor Author

reyoung commented Jan 17, 2023

Here is the PR for contributors go-python/license#20 .

@sbinet sbinet merged commit 5ef0384 into go-python:main Jan 17, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants