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

Better comment for the definition of the users #35

Merged
merged 1 commit into from
Jan 31, 2025
Merged

Conversation

btlogy
Copy link
Member

@btlogy btlogy commented Jan 31, 2025

Part of #29

Before we decide to avoid this approach.

Signed-off-by: Benoit Donneaux <benoit@leastauthority.com>
Copy link

ToFu - core

Step Outcome
✏️ Format success
🔧 Init success
🔍 Validate success
📄 Plan success
🛂 Verify success
👉 Result No changes. Your infrastructure matches the configuration.
show change(s)

show error(s)

Pusher: @btlogy, Action: pull_request
Workflow: tahoe-lafs/infrastructure/.github/workflows/tf-core.yml@refs/pull/35/merge

@btlogy btlogy marked this pull request as ready for review January 31, 2025 14:00
@btlogy btlogy merged commit 61cf4da into main Jan 31, 2025
1 check passed
@btlogy
Copy link
Member Author

btlogy commented Jan 31, 2025

Only commenting: merging w/o approval.

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.

1 participant