Uploaded image for project: 'Puppet Development Kit (moved to puppet.atlassian.net)'
  1. Puppet Development Kit (moved to puppet.atlassian.net)
  2. PDK-709

Update pdk-template CI configs to use PDK

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Duplicate
    • None
    • None
    • None
    • None
    • Hide
      • PDK-1501 is complete
      • Move the work that was previously in a branch on pdk-templates back into master
      • Revert the template url/ref changes that were done in PDK-1501
      • Ticket the work to change the sync.yml files for all modules audited in PDK-1499
      Show
      PDK-1501 is complete Move the work that was previously in a branch on pdk-templates back into master Revert the template url/ref changes that were done in PDK-1501 Ticket the work to change the sync.yml files for all modules audited in PDK-1499
    • Puppet Developer Experience
    • Needs Assessment

    Description

      There's been hints that lead towards there being a possible discrepancy between what the PDK runs during the 'validate' command, and the linting rules that are run on travis.

      Travis config tells travis to run the rake task which pulls in the supported modules rules from spec helper - The rules defined in puppetlabs spec helper currently panders to a lot of special cases for modules, and going forward we want those special cases to be removed so we are consistent with lint across the board.

      This sounds like it's possibly causing the validate command to pass, but when it comes to pushing a PR travis may fail on the additional cops ran.

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              helen Helen Campbell
              Votes:
              4 Vote for this issue
              Watchers:
              13 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Zendesk Support