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

[CSGI-2429 ] Add provider and a Standards datasource using Terraform Plugin Framework #787

Merged
merged 7 commits into from
Jan 30, 2024

Conversation

cjgajard
Copy link
Contributor

@cjgajard cjgajard commented Dec 15, 2023

Update go version to 1.20
Add pagerdutyplugin provider and Pagerduty/go-pagerduty as new api client
Move util functions to its own module
Add data source 'standards' to pagerdutyplugin/
Add acceptance test to standards datasource
Add website docs for pagerduty_standards

Depends on

@cjgajard cjgajard marked this pull request as draft December 15, 2023 15:14
@cjgajard cjgajard force-pushed the tpf-datasource-standards branch from 8ca4852 to 7c775a5 Compare December 15, 2023 15:48
@imjaroiswebdev imjaroiswebdev changed the title Add provider and a Standards datasource using Terraform Plugin Framework [CSGI-2429 ] Add provider and a Standards datasource using Terraform Plugin Framework Dec 18, 2023
@cjgajard cjgajard force-pushed the tpf-datasource-standards branch 2 times, most recently from f703b1d to db48975 Compare January 22, 2024 14:57
@cjgajard cjgajard marked this pull request as ready for review January 22, 2024 14:59
Copy link
Contributor

@imjaroiswebdev imjaroiswebdev left a comment

Choose a reason for hiding this comment

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

Before moving forward with merging there are a few comments to address @cjgajard. Please rebase and let know for second take.

main.go Outdated Show resolved Hide resolved
website/docs/d/standards.html.markdown Outdated Show resolved Hide resolved
website/docs/d/standards.html.markdown Outdated Show resolved Hide resolved
pagerdutyplugin/config.go Show resolved Hide resolved
pagerdutyplugin/config.go Outdated Show resolved Hide resolved
pagerdutyplugin/config.go Outdated Show resolved Hide resolved
pagerdutyplugin/config.go Outdated Show resolved Hide resolved
pagerdutyplugin/config.go Outdated Show resolved Hide resolved
pagerdutyplugin/data_source_pagerduty_standards.go Outdated Show resolved Hide resolved
Update go version to 1.20
Add pagerdutyplugin provider and new api client
Move util functions to its own module
Add data source 'standards' to pagerdutyplugin/
Add acceptance test to standards datasource
Add website docs for pagerduty_standards
@cjgajard cjgajard force-pushed the tpf-datasource-standards branch from db48975 to a23818c Compare January 29, 2024 21:37
@cjgajard cjgajard force-pushed the tpf-datasource-standards branch from a23818c to 54b8519 Compare January 29, 2024 21:54
@imjaroiswebdev imjaroiswebdev self-requested a review January 30, 2024 18:48
Copy link
Contributor

@imjaroiswebdev imjaroiswebdev left a comment

Choose a reason for hiding this comment

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

Great work @cjgajard! This is a very welcomed addition to the provider and its technical evolution going forward 🎉 💪🏽

@imjaroiswebdev imjaroiswebdev merged commit a9d0d02 into PagerDuty:master Jan 30, 2024
1 check passed
@cjgajard cjgajard deleted the tpf-datasource-standards branch February 23, 2024 17:12
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