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

chore: update service emails in pipeline #1263

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

chore: update service emails in pipeline #1263

wants to merge 1 commit into from

Conversation

publi0
Copy link
Collaborator

@publi0 publi0 commented Nov 18, 2024

What does this PR do?

This pull request includes minor changes to the .github/workflows/release.yaml file to update the email address used in the Git configuration.

Changes in Git configuration:

  • Updated the email address in the Git configuration from magalucloud.services@luizalabs.com to magalucloud.cli@luizalabs.com to ensure the correct email is used for commits. [1] [2]

@publi0 publi0 self-assigned this Nov 18, 2024
@publi0 publi0 requested a review from a team as a code owner November 18, 2024 18:34
@@ -117,7 +117,7 @@ jobs:
- name: Set up Git config
run: |
git config --global user.name "Magalu Cloud"
git config --global user.email "magalucloud.services@luizalabs.com"
git config --global user.email "magalucloud.cli@luizalabs.com"
git config --global commit.gpgsign true
git config --global user.signingkey $(gpg --list-secret-keys --keyid-format LONG | grep sec | awk '{print $2}' | cut -d'/' -f2)
Copy link
Member

Choose a reason for hiding this comment

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

creio que essa gpg vá quebrar...

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@luizcavalcanti Bora regerar esse cara

Copy link
Collaborator

Choose a reason for hiding this comment

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

Vamos precisar da chave privada desse novo usuário na runner da pipeline, e a chave pública no TF registry

@geffersonFerraz geffersonFerraz marked this pull request as draft November 19, 2024 18:32
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.

3 participants