Use dir instead of getmembers

[sethaxen]

Fixes #857 by using dir to get names of properties instead of getmembers, which also gets the properties themselves and therefore errors when properties are optional.

[codecov-io]

Codecov Report

Merging #879 (173542e) into master (ead312f) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #879      +/-   ##
==========================================
- Coverage   67.63%   67.61%   -0.02%     
==========================================
  Files          20       20              
  Lines        1965     1967       +2     
==========================================
+ Hits         1329     1330       +1     
- Misses        636      637       +1     

Flag	Coverage Δ
unittests	67.61% <100.00%> (-0.02%)	⬇️

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

Impacted Files	Coverage Δ
src/PyCall.jl	69.75% <100.00%> (+0.18%)	⬆️
src/conversions.jl	63.13% <0.00%> (-0.26%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ead312f...173542e. Read the comment docs.

[sethaxen]

@stevengj can a new release be tagged as well?

[sethaxen]

@stevengj can you tag a new release? One of my packages needs this fix.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.61%. Comparing base (ead312f) to head (173542e).
Report is 51 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #879      +/-   ##
==========================================
- Coverage   67.63%   67.61%   -0.02%     
==========================================
  Files          20       20              
  Lines        1965     1967       +2     
==========================================
+ Hits         1329     1330       +1     
- Misses        636      637       +1     

Flag	Coverage Δ
unittests	67.61% <100.00%> (-0.02%)	⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.