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

corrected the title by making the variable name plural #404

Merged
merged 1 commit into from
Feb 10, 2021
Merged

corrected the title by making the variable name plural #404

merged 1 commit into from
Feb 10, 2021

Conversation

crosbymichael1
Copy link
Contributor

@crosbymichael1 crosbymichael1 commented Feb 5, 2021

Description

corrected the title by making the variable name plural
from mongodbatlas_network_peering to mongodbatlas_network_peerings

Link to any related issue(s):

Type of change:

  • Documentation fix/enhancement

  • signed mongodb contributing agreement

corrected the title by making the variable name plural
from mongodbatlas_network_peering to mongodbatlas_network_peerings
Copy link
Collaborator

@themantissa themantissa left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you for taking the time to fix these small, but important, items!

@themantissa themantissa self-requested a review February 8, 2021 23:16
@themantissa
Copy link
Collaborator

@crosbymichael1 before I can merge I need to verify you've signed our CLA, want to check that you've already done so? Link can be found here: https://github.com/mongodb/terraform-provider-mongodbatlas/blob/master/CONTRIBUTING.md

@crosbymichael1
Copy link
Contributor Author

@themantissa I have signed the CLA, thank you.

Copy link
Collaborator

@themantissa themantissa left a comment

Choose a reason for hiding this comment

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

LGTM! CLA confirmed.

@themantissa themantissa merged commit 5720260 into mongodb:master Feb 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants