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

Replace incorrect spelling of COMMITER with COMMITTER #192

Merged
merged 1 commit into from
Dec 15, 2022

Conversation

hisaac
Copy link
Contributor

@hisaac hisaac commented Dec 14, 2022

Checklist

  • I've read and followed the Contribution Guidelines
  • step.yml and README.md is updated with the changes (if needed)

Version

Version update will occur when next is merged in at the end.

Context

I noticed that we had 2 environment variables with a typo in them. This fixes that.

Changes

  • Replace GIT_CLONE_COMMIT_COMMITER_NAME with GIT_CLONE_COMMIT_COMMITTER_NAME
  • Replace GIT_CLONE_COMMIT_COMMITER_EMAIL with GIT_CLONE_COMMIT_COMMITTER_EMAIL

@hisaac hisaac requested a review from ofalvai December 14, 2022 17:08
@hisaac
Copy link
Contributor Author

hisaac commented Dec 14, 2022

@ofalvai What do you think of this change? I figure as long as we're doing a breaking change anyway, it couldn't hurt to fix this typo in the environment variable.

Comment on lines +78 to +87
| Environment Variable | Description |
|------------------------------------|------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
| `GIT_CLONE_COMMIT_HASH` | SHA hash of the checked-out commit. |
| `GIT_CLONE_COMMIT_MESSAGE_SUBJECT` | Commit message of the checked-out commit. |
| `GIT_CLONE_COMMIT_MESSAGE_BODY` | Commit message body of the checked-out commit. |
| `GIT_CLONE_COMMIT_COUNT` | Commit count after checkout. Count will only work properly if no `--depth` option is set. If `--depth` is set then the history truncated to the specified number of commits. Count will **not** fail but will be the clone depth. |
| `GIT_CLONE_COMMIT_AUTHOR_NAME` | Author of the checked-out commit. |
| `GIT_CLONE_COMMIT_AUTHOR_EMAIL` | Email of the checked-out commit. |
| `GIT_CLONE_COMMIT_COMMITTER_NAME` | Committer name of the checked-out commit. |
| `GIT_CLONE_COMMIT_COMMITTER_EMAIL` | Email of the checked-out commit. |
Copy link
Contributor Author

Choose a reason for hiding this comment

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

All these extra spaces are courtesy of GoLand, which added them automatically to normalize the Markdown table.

Copy link
Contributor

@ofalvai ofalvai left a comment

Choose a reason for hiding this comment

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

Yeah, it's probably a good idea to rip the bandaid off now and fix all of these. Thank you and let me know if you find more, I missed this!

@hisaac hisaac merged commit 0c260de into next Dec 15, 2022
@hisaac hisaac deleted the fix-environment-variable-typo branch December 15, 2022 14:40
@ofalvai ofalvai mentioned this pull request Jan 2, 2023
2 tasks
ofalvai pushed a commit that referenced this pull request Jan 3, 2023
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.

2 participants