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

Release v0.1.69 #558

Merged
merged 3 commits into from
Sep 29, 2023
Merged

Conversation

tirthct
Copy link
Contributor

@tirthct tirthct commented Sep 29, 2023

-447488d Added default values for CIDRs and host prefix -672ca8b Fixed CIDR order and added back the comment -fbff80f Fixed the comment statement
-1f2b39b fix duplicate machine pool information printed for the same cluster -f0fadd8 add tests for list machinepools command
-5b60bbf added failure tests for list machinepools command -498c54e Filtered OCM versions for marketplace gcp clusters -a9f32b7 Changed string pointer to string and removed nil check -0aca318 Changed gcp-marketplace keyword to marketplace-gcp

-447488d Added default values for CIDRs and host prefix
-672ca8b Fixed CIDR order and added back the comment
-fbff80f Fixed the comment statement
-1f2b39b fix duplicate machine pool information printed for the same cluster
-f0fadd8 add tests for list machinepools command
-5b60bbf added failure tests for list machinepools command
-498c54e Filtered OCM versions for marketplace gcp clusters
-a9f32b7 Changed string pointer to string and removed nil check
-0aca318 Changed gcp-marketplace keyword to marketplace-gcp
CHANGES.md Outdated
Comment on lines 14 to 16
-498c54e Filtered OCM versions for marketplace gcp clusters
-a9f32b7 Changed string pointer to string and removed nil check
-0aca318 Changed gcp-marketplace keyword to marketplace-gcp
Copy link
Contributor

Choose a reason for hiding this comment

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

looks like the previous PR got merged without squashing. Can you combine them in something more representative?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changed and kept meaningful commit logs in changes.md file

@cristianoveiga cristianoveiga merged commit d9b1346 into openshift-online:main Sep 29, 2023
4 checks passed
@tirthct tirthct deleted the release_0.1.69 branch September 29, 2023 15:27
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