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.

Sign up for GitHub

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

Jump to bottom

Add my GitHub id #32

Merged
merged 2 commits into from
Dec 29, 2021
Merged

Add my GitHub id #32

merged 2 commits into from
Dec 29, 2021

Conversation

sozysozbot
Copy link
Contributor

No description provided.

@sksat
Copy link
Contributor

sksat commented Dec 26, 2021

おや，terraform plan走らんなー

@sksat sksat mentioned this pull request Dec 26, 2021
Closed
@sksat
Copy link
Contributor

sksat commented Dec 28, 2021

@sozysozbot
#41 で対応したのでrebaseお願いします

@sksat
Copy link
Contributor

sksat commented Dec 28, 2021

それとももしかして今ラベル付けても動くのかな

@sksat
Copy link
Contributor

sksat commented Dec 28, 2021

あ，動いた

@github-actions
Copy link

To view this run in a browser, visit:
https://app.terraform.io/app/sksat/yohanesu-infra/runs/run-wVeyKNVjc6s9r1eE

@sksat
Copy link
Contributor

sksat commented Dec 28, 2021

これ無のplanが行われてるな

@sksat
Copy link
Contributor

sksat commented Dec 28, 2021

これmainがcheckoutされとるやないかい

@sksat
Copy link
Contributor

sksat commented Dec 28, 2021 •
edited
Loading

ウーム

@sksat sksat mentioned this pull request Dec 28, 2021
Merged
2 tasks
@github-actions
Copy link

To view this run in a browser, visit:
https://app.terraform.io/app/sksat/yohanesu-infra/runs/run-wqMG4vdxSQGengtW

@sksat
Copy link
Contributor

sksat commented Dec 29, 2021

とりあえずこれでOKだな

sksat
sksat requested changes Dec 29, 2021
View reviewed changes
terraform/github.tf Outdated Show resolved Hide resolved
@sozysozbot @sksat
Add a trailing comma
7808128 
Co-authored-by: sksat <sksat@sksat.net>
@github-actions
Copy link

To view this run in a browser, visit:
https://app.terraform.io/app/sksat/yohanesu-infra/runs/run-Qcy9FbJd86VZef3M

sksat
sksat approved these changes Dec 29, 2021
View reviewed changes
Copy link
Contributor

@sksat sksat left a comment

Choose a reason for hiding this comment

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

LGTM

@sksat sksat merged commit 768b0b1 into yohane-su:main Dec 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sksat sksat sksat approved these changes

Assignees
No one assigned
Labels
ok to plan
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sozysozbot @sksat