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

implement missing maintenance-window subcommands #363

Merged

Conversation

Hsn723
Copy link
Contributor

@Hsn723 Hsn723 commented Sep 13, 2021

The pd maintenance-window command was missing subcommands other than create, so I went ahead and implemented them. create is also using a deprecated library function so I also addressed that.

Copy link
Collaborator

@theckman theckman left a comment

Choose a reason for hiding this comment

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

Thank you for the contribution. I did request a few changes.

command/maintenance_window_delete.go Outdated Show resolved Hide resolved
command/maintenance_window_delete.go Show resolved Hide resolved
command/maintenance_window_list.go Outdated Show resolved Hide resolved
command/maintenance_window_delete.go Show resolved Hide resolved
command/maintenance_window_list.go Show resolved Hide resolved
command/maintenance_window_create.go Outdated Show resolved Hide resolved
command/maintenance_window_update.go Outdated Show resolved Hide resolved
@theckman theckman added this to the v1.5.0 milestone Sep 28, 2021
@Hsn723 Hsn723 force-pushed the implement-maintenance-window-command branch from 25b01e0 to b3cc824 Compare September 29, 2021 00:10
@Hsn723
Copy link
Contributor Author

Hsn723 commented Sep 29, 2021

@theckman
Thank you for the review. I have applied the requested changes. Please take a look

Copy link
Collaborator

@theckman theckman left a comment

Choose a reason for hiding this comment

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

Thank you for your contribution, as well as working with me on the requested changes. It'll probably still take a little bit of time until the v1.5.0 tag is cut on the repo, but within the next few minutes your changes should be available on the master branch if you'd like to start making use of them.

@theckman theckman merged commit 704beda into PagerDuty:master Oct 1, 2021
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.

None yet

2 participants