Skip to content

Latest commit

 

History

History
executable file
·
139 lines (90 loc) · 7 KB

guide-opening-a-pr.md

File metadata and controls

executable file
·
139 lines (90 loc) · 7 KB

Opening a PR

Firstly all contributions are welcome!

There is no change too small for us to accept and minor formatting, consistency and documentation PRs are very welcome! However, before making any large or structural changes it is recommended to seek feedback (preferably by reaching out in our community slack) to prevent wasted time and effort. We may already be working on a solution, or have a different direction we would like to take.

If you are ever unsure please just reach out, we are more than happy to guide you in the right direction!

Considerations

As a general rule, the smaller the PR the quicker it's merged - as such when upgrading an SDK and introducing new properties we'd ask that you split that into multiple smaller PR's, for example if you were planning on updating an SDK to add a new resource and update an existing one we would prefer 3 separate PRs:

  1. Update the Cosmos DB SDK to use API Version 2022-02-02 from 2020-01-01.
  2. Add the new property new_feature to the azurerm_cosmosdb_* resources.
  3. Introduce the New Resource azurerm_cosmosdb_resource.

We also recommend not opening a PR based on your main branch. By doing this any changed pushed to the PR may inadvertently be also pushed to your main branch without warning.

Due to the high volume of PRs on the project and to ensure maintainers are able to focus on changes which are ready to review, please do not open Draft PRs or work that is not yet ready to be reviewed.

Process

Pull Requests generally go through a number of phases which vary slightly depending on what's being changed.

The following guides cover the more common scenarios we see:

In general, Pull Requests which add/change either code or SDK's go through the following steps:

  1. Make / commit the changes.
  2. Run GitHub Actions linting and checks locally with the make command make pr-check.
  3. Run all relevant Acceptance Tests.
  4. Open a Pull Request (see below on What makes a good PR?).
  5. GitHub actions will trigger and run all linters.
  6. A Maintainer will review the PR and also run acceptance tests against our test subscription.
  7. Once all comments have been addressed and tests pass the PR will be merged
  8. The maintainer will update the CHANGELOG.md.

What makes a good PR?

  • Don't send the PR from your main branch.
  • The PR Title is obvious/clear about what it's changing (see Title below).
  • The PR Body contains a summary of what/why is included (see Body below).
  • any linked Issues (see Body below)

Title

The title of the PR should clearly state what the PR is doing, and ideally should match the entry that will end up in the changelog.

Examples of good PR titles:

  • azurerm_storage_management_policy - Mark rule.filters.blob_type as required
  • azurerm_container_registry - support updating replications on demand
  • azurerm_automation_account - support for the encrytion, local_authentication_enabled, and tags properties
  • Data Source: azurerm_automation_account - prevent panic (#15474) by adding a nil check
  • Upgrade bot API version from 2021-03-01 to 2021-05-01-preview
  • New Resource: azurerm_managed_disk_sas_token
  • New Data Source: azurerm_managed_disk_sas_token
  • Docs: Fix wrong command in 3.0-upgrade-guide

Examples of poorly written PR titles:

  • fix sql bug
  • fixes #1234
  • new resource
  • upgrade sdk
  • upgrade compute api
  • add cosmos property
  • support encryption, local_authentication_enabled properties

Body

An example of our PR template is shown below.

Community Note

  • Please vote on this PR by adding a 👍 reaction to the original PR to help the community and maintainers prioritize for review
  • Please do not leave "+1" or "me too" comments, they generate extra noise for PR followers and do not help prioritize for review

PR Checklist

  • Have you followed the guidelines in our Contributing Documentation?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you used a meaningful PR description to help maintainers and other users understand this change and help prevent duplicate work? Example: “resource_name_here - description of change e.g. adding property new_property_name_here
  • Do your changes close any open issues? If so please include appropriate closing keywords below.

New Feature Submissions

  • Does your submission include Test coverage as described in the Contribution Guide and the tests pass? (if this is not possible for any reason, please include details of why below)

Changes to existing Resource / Data Source

  • Have you added an explanation of what your changes do and why you'd like us to include them? (This may be covered by linking to an issue above, but may benefit from additional explanation)
  • Have you written new tests for your resource or datasource changes?
  • Have you successfully run tests with your changes locally? If not, please provide details on testing challenges that prevented you running the tests.

Documentation Changes

  • Documentation is written in International English.
  • Documentation is written in a helpful and kind way to assist users that may be unfamiliar with the resource / data source.

Description

Related Issue(s)

Use linking keywords here like "fixes", "closes", "resolves", etc:

Fixes #1234, fixes #5678, fixes #9101

Change Log

Changelog Format

  • azurerm_resource - support for the thing1 property [GH-00000]
  • Bug Fix
  • New Feature

[!NOTE] If this PR changes meaningfully during the course of review please update the title and description as required.