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

fix: use commit author for project creator #715

Merged
merged 2 commits into from Oct 2, 2019

Conversation

rokroskar
Copy link
Member

The PR corrects the wrong behavior that the user executing the git command was being set as the project creator. Instead, if the project creator does not exist, the git history of .renku/metadata.yml is searched for the first commit and its author is used as the project creator.

closes #713

@rokroskar rokroskar requested a review from a team as a code owner September 26, 2019 13:55
@rokroskar rokroskar force-pushed the 713-wrong-project-creator branch 2 times, most recently from d1f0d05 to 9a43b0e Compare September 30, 2019 09:31
@rokroskar rokroskar changed the title [wip] fix: use commit author for project creator fix: use commit author for project creator Sep 30, 2019
@rokroskar rokroskar force-pushed the 713-wrong-project-creator branch 2 times, most recently from 603f091 to 0b57195 Compare September 30, 2019 13:46
renku/models/datasets.py Outdated Show resolved Hide resolved
tests/test_dataset.py Outdated Show resolved Hide resolved
tests/test_projects.py Outdated Show resolved Hide resolved
Copy link
Contributor

@jsam jsam 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 looking into this.

renku/api/repository.py Outdated Show resolved Hide resolved
renku/api/repository.py Outdated Show resolved Hide resolved
renku/api/repository.py Outdated Show resolved Hide resolved
tests/test_dataset.py Outdated Show resolved Hide resolved
@rokroskar rokroskar force-pushed the 713-wrong-project-creator branch 2 times, most recently from 4a09800 to d7b9261 Compare October 2, 2019 11:38
@rokroskar rokroskar merged commit 1a40ebe into master Oct 2, 2019
@rokroskar rokroskar deleted the 713-wrong-project-creator branch October 2, 2019 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

renku log returns the wrong creator of Project
3 participants