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

Git version #22745

Merged
merged 4 commits into from
Sep 19, 2018
Merged

Git version #22745

merged 4 commits into from
Sep 19, 2018

Conversation

alimcmaster1
Copy link
Member

Finished off the work largely completed in this pull request: #21680

@pep8speaks
Copy link

Hello @alimcmaster1! Thanks for submitting the PR.

@@ -185,7 +185,7 @@ Other Enhancements
- :class:`Resampler` now is iterable like :class:`GroupBy` (:issue:`15314`).
- :meth:`Series.resample` and :meth:`DataFrame.resample` have gained the :meth:`Resampler.quantile` (:issue:`15023`).
- :meth:`Index.to_frame` now supports overriding column name(s) (:issue:`22580`).

- New attribute :attr:`__git_version__` will return git commit sha of current build (:issue:`21295` ).
Copy link
Member

Choose a reason for hiding this comment

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

small nitpick: extra space before the closing parenthesis in (:issue:`21295` )

Copy link
Member Author

Choose a reason for hiding this comment

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

updated

@jschendel jschendel added the Build Library building on various platforms label Sep 18, 2018
@codecov
Copy link

codecov bot commented Sep 18, 2018

Codecov Report

Merging #22745 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #22745   +/-   ##
=======================================
  Coverage   92.16%   92.16%           
=======================================
  Files         169      169           
  Lines       50769    50769           
=======================================
  Hits        46791    46791           
  Misses       3978     3978
Flag Coverage Δ
#multiple 90.57% <ø> (ø) ⬆️
#single 42.32% <ø> (ø) ⬆️

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 1a12c41...b5f7048. Read the comment docs.

@bashtage
Copy link
Contributor

  • [y] whatsnew entry

Not that is matters, but if you use an [x] instead of [y] you get a tick mark.

  • Item 1
  • Item 2
  • [y] Item 3

@TomAugspurger
Copy link
Contributor

LGTM, thanks!

@TomAugspurger TomAugspurger merged commit 9f4ccb5 into pandas-dev:master Sep 19, 2018
@jorisvandenbossche jorisvandenbossche added this to the 0.24.0 milestone Sep 19, 2018
aeltanawy pushed a commit to aeltanawy/pandas that referenced this pull request Sep 20, 2018
Sup3rGeo pushed a commit to Sup3rGeo/pandas that referenced this pull request Oct 1, 2018
@alimcmaster1 alimcmaster1 deleted the git-version branch October 9, 2018 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build Library building on various platforms
Projects
None yet
Development

Successfully merging this pull request may close these issues.

access git revision of currently running pandas
6 participants