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

feat: Expose git author parameters #202

Merged
merged 4 commits into from
Aug 8, 2023
Merged

Conversation

christiansiegel
Copy link
Member

@christiansiegel christiansiegel commented May 8, 2023

Add optional git author parameters to all commands creating git commits.
If no author is provided the git user and email will be used for both committer and author.

Example:

gitopscli deploy \
  --git-user "GitOps CLI" \
  --git-email "[email protected]" \
  --git-author-name "John Doe" \
  --git-author-email "[email protected]" \
  ...

Resolves #195

Add optional co-author parameters to all commands creating git commits.
The co-author will be added as trailer at the end of the commit message.

Example:
```bash
gitopscli deploy \
  --co-author-name 'John Doe' \
  --co-author-email [email protected] \
  ...
```

```
changed 'backend.tag' to 'foo' in values.yaml

Co-authored-by: John Doe <[email protected]>
```

Resolved #195
gitopscli/git_api/git_repo.py Show resolved Hide resolved
@christiansiegel christiansiegel changed the title feat: Add optional co-author parameters feat: Expose git author parameters Aug 7, 2023
@christiansiegel christiansiegel self-assigned this Aug 7, 2023
Copy link

@mikevader mikevader left a comment

Choose a reason for hiding this comment

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

Great work. Thanks a lot.

@christiansiegel christiansiegel merged commit d61deda into master Aug 8, 2023
2 checks passed
@github-actions
Copy link

github-actions bot commented Aug 8, 2023

🎉 This PR is included in version 5.8.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@christiansiegel christiansiegel deleted the feat/co-author branch August 8, 2023 05:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide additional author with parameter
2 participants