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

test: require 100% unit test coverage (via synth) #359

Merged
merged 7 commits into from
Apr 14, 2021
Merged

Commits on Apr 13, 2021

  1. changes without context

            autosynth cannot find the source of changes triggered by earlier changes in this
            repository, or by version upgrades to tools such as linters.
    yoshi-automation committed Apr 13, 2021
    Configuration menu
    Copy the full SHA
    e8bfe5f View commit details
    Browse the repository at this point in the history
  2. feat: add RPC Priority request options

    PiperOrigin-RevId: 364449524
    
    Source-Author: Google APIs <noreply@google.com>
    Source-Date: Mon Mar 22 17:39:37 2021 -0700
    Source-Repo: googleapis/googleapis
    Source-Sha: 6598bb829c9e9a534be674649ffd1b4671a821f9
    Source-Link: googleapis/googleapis@6598bb8
    yoshi-automation committed Apr 13, 2021
    Configuration menu
    Copy the full SHA
    eab33e9 View commit details
    Browse the repository at this point in the history
  3. chore(deps): update precommit hook pycqa/flake8 to v3.9.0

    [![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)
    
    This PR contains the following updates:
    
    | Package | Type | Update | Change |
    |---|---|---|---|
    | [pycqa/flake8](https://gitlab.com/pycqa/flake8) | repository | minor | `3.8.4` -> `3.9.0` |
    
    ---
    
    ### Release Notes
    
    <details>
    <summary>pycqa/flake8</summary>
    
    ### [`v3.9.0`](https://gitlab.com/pycqa/flake8/compare/3.8.4...3.9.0)
    
    [Compare Source](https://gitlab.com/pycqa/flake8/compare/3.8.4...3.9.0)
    
    </details>
    
    ---
    
    ### Renovate configuration
    
    :date: **Schedule**: At any time (no schedule defined).
    
    :vertical_traffic_light: **Automerge**: Disabled by config. Please merge this manually once you are satisfied.
    
    :recycle: **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
    
    :no_bell: **Ignore**: Close this PR and you won't be reminded about this update again.
    
    ---
    
     - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box
    
    ---
    
    This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/synthtool).
    
    Source-Author: WhiteSource Renovate <bot@renovateapp.com>
    Source-Date: Tue Mar 23 17:38:03 2021 +0100
    Source-Repo: googleapis/synthtool
    Source-Sha: f5c5904fb0c6aa3b3730eadf4e5a4485afc65726
    Source-Link: googleapis/synthtool@f5c5904
    yoshi-automation committed Apr 13, 2021
    Configuration menu
    Copy the full SHA
    382db5e View commit details
    Browse the repository at this point in the history
  4. test(python): use constraints files to check dependency lower bounds

    Use a constraints file when installing dependencies for system and unit tests nox sessions.
    
    https://pip.pypa.io/en/stable/user_guide/#constraints-files
    > Constraints files are requirements files that **only control which version of a requirement is installed, not whether it is installed or not**. Their syntax and contents is nearly identical to Requirements Files. There is one key difference: Including a package in a constraints file does not trigger installation of the package.
    
    ```
    testing
    ├── constraints-3.10.txt
    ├── constraints-3.11.txt
    ├── constraints-3.6.txt
    ├── constraints-3.7.txt
    ├── constraints-3.8.txt
    └── constraints-3.9.txt
    ```
    
    Going forward, one constraints file (currently 3.6) will be populated with every library requirement and extra listed in the `setup.py`. The constraints file will pin each requirement to the lower bound. This ensures that library maintainers will see test failures if they forget to update a lower bound on a dependency.
    
    See googleapis/python-bigquery#263 for an example
    
    Source-Author: Bu Sun Kim <8822365+busunkim96@users.noreply.github.com>
    Source-Date: Tue Mar 23 10:52:02 2021 -0600
    Source-Repo: googleapis/synthtool
    Source-Sha: 86ed43d4f56e6404d068e62e497029018879c771
    Source-Link: googleapis/synthtool@86ed43d
    yoshi-automation committed Apr 13, 2021
    Configuration menu
    Copy the full SHA
    e07459d View commit details
    Browse the repository at this point in the history
  5. build(python): update docfx job to use new plugin

    Source-Author: Dan Lee <71398022+dandhlee@users.noreply.github.com>
    Source-Date: Tue Mar 30 19:36:37 2021 -0400
    Source-Repo: googleapis/synthtool
    Source-Sha: 4501974ad08b5d693311457e2ea4ce845676e329
    Source-Link: googleapis/synthtool@4501974
    yoshi-automation committed Apr 13, 2021
    Configuration menu
    Copy the full SHA
    5d96258 View commit details
    Browse the repository at this point in the history
  6. chore: Add license headers for python config files

    Source-Author: Anthonios Partheniou <partheniou@google.com>
    Source-Date: Tue Apr 6 11:32:03 2021 -0400
    Source-Repo: googleapis/synthtool
    Source-Sha: 5b5bf6d519b2d658d9f2e483d9f6f3d0ba8ee6bc
    Source-Link: googleapis/synthtool@5b5bf6d
    yoshi-automation committed Apr 13, 2021
    Configuration menu
    Copy the full SHA
    f893d25 View commit details
    Browse the repository at this point in the history
  7. chore: add constraints file check for python samples

    This is the sibling PR to GoogleCloudPlatform/python-docs-samples#5611
    and this is the issue opened for it GoogleCloudPlatform/python-docs-samples#5549
    
    If you look at the files in [this example repo](https://github.com/leahecole/testrepo-githubapp/pull/31/files), you'll see that renovate successfully opened a PR on three constraints files in `samples` directories and subdirectories, and properly ignored `constraints` files at the root level
    
    cc @tswast
    
    TODO:
    
    - [x]   update renovate to check for samples/constraints.txt dependency updates
    - [x]  run lint locally to double check that I'm not introducing lint error
    
    Source-Author: Leah E. Cole <6719667+leahecole@users.noreply.github.com>
    Source-Date: Fri Apr 9 22:50:04 2021 -0700
    Source-Repo: googleapis/synthtool
    Source-Sha: 0a071b3460344886297a304253bf924aa68ddb7e
    Source-Link: googleapis/synthtool@0a071b3
    yoshi-automation committed Apr 13, 2021
    Configuration menu
    Copy the full SHA
    6804e87 View commit details
    Browse the repository at this point in the history