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.

Sign up for GitHub

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

Jump to bottom

Remove Pulp 2 functionality #271

Closed
wants to merge 4 commits into from
Closed

Remove Pulp 2 functionality #271

wants to merge 4 commits into from

Commits on Dec 2, 2020

  1. Remove coverage spec 

    This is handled in the spec helper and only leads to a duplicate report.
    @ekohl
    ekohl committed Dec 2, 2020
    Configuration menu
    Copy the full SHA
    a9f45dd View commit details
    Browse the repository at this point in the history

  2. Always set pulpcore_enabled in specs 

    When Pulp 2 parameters are set, Pulp 3 parameters should also be set.
This disabled Pulp 3 in many cases leading to faster spec tests.
    @ekohl
    ekohl committed Dec 2, 2020
    Configuration menu
    Copy the full SHA
    00b822d View commit details
    Browse the repository at this point in the history

  3. Use RSpec's context nesting 

    With RSpec contexts can be nested and they'll inherit the parent let
blocks. The descriptions will be included as well.
    @ekohl
    ekohl committed Dec 2, 2020
    Configuration menu
    Copy the full SHA
    00992c7 View commit details
    Browse the repository at this point in the history

  4. Remove Pulp 2 functionality

    @ekohl
    ekohl committed Dec 2, 2020
    Configuration menu
    Copy the full SHA
    ca9661c View commit details
    Browse the repository at this point in the history