Skip to content

refactor: migrate resource_github_actions_environment_variable to tflog#3477

Open
texasich wants to merge 1 commit into
integrations:mainfrom
texasich:refactor/tflog-actions-env-variable
Open

refactor: migrate resource_github_actions_environment_variable to tflog#3477
texasich wants to merge 1 commit into
integrations:mainfrom
texasich:refactor/tflog-actions-env-variable

Conversation

@texasich
Copy link
Copy Markdown

@texasich texasich commented Jun 5, 2026

Replace Go standard log package with HashiCorp structured tflog for consistent logging across the provider.

Changes:

  • Replace log import with tflog import
  • Convert log.Printf to tflog.Info with structured fields

Resolves #3070 (partial — this PR covers one file)

This is a resubmission of #3325 (was closed due to inactivity / capacity).


Before the change?

resource_github_actions_environment_variable.go uses Go's standard log.Printf for logging, which produces unstructured output inconsistent with the rest of the provider.

After the change?

Logging uses HashiCorp's tflog.Info with structured key-value fields (variable_id), consistent with the pattern established in other migrated files like resource_github_membership.go.

Pull request checklist

  • Schema migrations have been created if needed
  • Acceptance tests have been run (no behavioral changes — logging only)
  • Documentation has been updated if needed (N/A — internal refactor)

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 5, 2026

👋 Hi, and thank you for this contribution!

This repo is maintained by GitHub and community members on a best-effort basis. We'll get to this as soon as we can.

You can help us prioritize by joining the discussion on open issues and PRs, sharing details on the changes you need, and reviewing other contributions.


🤖 This is an automated message.

@deiga
Copy link
Copy Markdown
Collaborator

deiga commented Jun 5, 2026

Please reopen #3325 instead of creating a new PR

@texasich
Copy link
Copy Markdown
Author

texasich commented Jun 5, 2026

@deiga — closing this duplicate per your request. Reopening #3325 (the original PR) now.

Replace Go standard `log` package with HashiCorp structured `tflog` for consistent logging across the provider.

Changes:
- Replace `log` import with tflog import
- Convert `log.Printf` to `tflog.Info` with structured fields

Part of integrations#3070

Closes integrations#3325

Signed-off-by: texasich <101962694+texasich@users.noreply.github.com>
@texasich texasich force-pushed the refactor/tflog-actions-env-variable branch from 6c981b9 to 2ed3a2d Compare June 5, 2026 23:06
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.

[MAINT] Tracking issue for refactoring logging to use tflog instead of log

2 participants