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

access git revision from inside the project #72

Closed
jangorecki opened this issue Aug 17, 2018 · 2 comments · Fixed by #73
Closed

access git revision from inside the project #72

jangorecki opened this issue Aug 17, 2018 · 2 comments · Fixed by #73
Labels
good first issue 🔰 Good for newcomers

Comments

@jangorecki
Copy link

modin.__git_revision__ could return full git SHA of latest commit.
It is useful for developers to track which particular revision they are running at the moment.
Something like dask/dask#1760

@devin-petersohn devin-petersohn added pandas concordance 🐼 Functionality that does not match pandas good first issue 🔰 Good for newcomers and removed pandas concordance 🐼 Functionality that does not match pandas labels Aug 17, 2018
@devin-petersohn
Copy link
Collaborator

This would be great to add. If you are able and willing to contribute this, that would be great. Otherwise, I am happy to add this in the next couple of days.

@simon-mo
Copy link
Collaborator

simon-mo commented Aug 1, 2019

@devin-petersohn Did this get removed from the codebase some reason? Because user from pip install will always get Unknown?

dchigarev pushed a commit to dchigarev/modin that referenced this issue Aug 25, 2020
We need to disable lazy fetch since there are some problems with our benchmarks
mdatre added a commit to mdatre/modin that referenced this issue Feb 21, 2023
…project#72)

also revert `_check_index_name` change from 6c82427

[please upstream to modin-project/modin]
vnlitvinov pushed a commit to vnlitvinov/modin that referenced this issue Feb 27, 2023
…project#72)

also revert `_check_index_name` change from 6c82427

[please upstream to modin-project/modin]
vnlitvinov pushed a commit to vnlitvinov/modin that referenced this issue Mar 16, 2023
…project#72)

also revert `_check_index_name` change from 6c82427

[please upstream to modin-project/modin]
vnlitvinov pushed a commit to vnlitvinov/modin that referenced this issue Mar 16, 2023
…project#72)

also revert `_check_index_name` change from 6c82427

[please upstream to modin-project/modin]
vnlitvinov pushed a commit to vnlitvinov/modin that referenced this issue Mar 16, 2023
…project#72)

also revert `_check_index_name` change from 6c82427

[please upstream to modin-project/modin]
vnlitvinov pushed a commit to vnlitvinov/modin that referenced this issue Apr 12, 2023
…project#72)

also revert `_check_index_name` change from 6c82427

[please upstream to modin-project/modin]
vnlitvinov pushed a commit to vnlitvinov/modin that referenced this issue Apr 13, 2023
…project#72)

also revert `_check_index_name` change from 6c82427

[please upstream to modin-project/modin]
vnlitvinov pushed a commit to vnlitvinov/modin that referenced this issue Apr 17, 2023
…project#72)

also revert `_check_index_name` change from 6c82427

[please upstream to modin-project/modin]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue 🔰 Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants