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

(FM-8081) pdk new transport #666

Merged
merged 2 commits into from
Jul 1, 2019

Conversation

DavidS
Copy link
Contributor

@DavidS DavidS commented May 17, 2019

@coveralls
Copy link

coveralls commented May 17, 2019

Coverage Status

Coverage decreased (-1.0%) to 92.088% when pulling 8b11d9c on DavidS:fm-8081-pdk-new-transport into 49623a3 on puppetlabs:master.

@DavidS
Copy link
Contributor Author

DavidS commented May 20, 2019

Acceptance tests fail expectedly as they need the puppetlabs/pdk-templates#251 merged first

@scotje
Copy link
Contributor

scotje commented May 21, 2019

Temporarily blocked on at least adding the resource API gem to the PDK package and come up with some solution to the rspec-mocks stuff.

@DavidS
Copy link
Contributor Author

DavidS commented May 22, 2019

Added puppetlabs/puppet-module-gems#98 to make the Resource API gem part of the default bundle.

I have no simpler solution for the rspec-mocks stuff than is already on the table; specifically

  • modifying the template renderer to lay down a .sync.yml with the default
  • have pdk validate nag people on requesting a specific mocking library if the module hasn't chosen one, and nag them more if they've chosen mocha

@DavidS DavidS force-pushed the fm-8081-pdk-new-transport branch from c291d76 to 38c60fb Compare May 30, 2019 12:35
@DavidS
Copy link
Contributor Author

DavidS commented Jun 17, 2019

restarted travis and appveyor tests since everything else has been merged by now

@DavidS DavidS force-pushed the fm-8081-pdk-new-transport branch from 38c60fb to bfb384b Compare June 17, 2019 10:37
@DavidS
Copy link
Contributor Author

DavidS commented Jun 17, 2019

... and fixed the code to pass unit tests. not to break other code bits.

This change implements a `pdk new transport` command for
using the new Resource API transports. It is closely
modelled on the `pdk new provider` command.
@DavidS DavidS force-pushed the fm-8081-pdk-new-transport branch from bfb384b to 8b11d9c Compare June 17, 2019 12:32
@DavidS DavidS requested a review from scotje June 17, 2019 13:52
@DavidS DavidS added the feature label Jun 17, 2019
@DavidS
Copy link
Contributor Author

DavidS commented Jun 18, 2019

@scotje this is now complete as far as I can tell.

@tphoney
Copy link
Contributor

tphoney commented Jun 18, 2019

🔥 👿

@DavidS DavidS changed the title (FM- 8081) pdk new transport (FM-8081) pdk new transport Jul 1, 2019
@DavidS DavidS merged commit e346560 into puppetlabs:master Jul 1, 2019
@DavidS DavidS deleted the fm-8081-pdk-new-transport branch July 1, 2019 16:32
DavidS added a commit to DavidS/pdk that referenced this pull request Jul 1, 2019
…-transport"

This reverts commit e346560, reversing
changes made to 5e0e168.
scotje added a commit that referenced this pull request Jul 1, 2019
Revert "Merge pull request #666 from DavidS/fm-8081-pdk-new-transport"
logicminds pushed a commit to nwops/pdk that referenced this pull request Nov 30, 2020
…-transport"

This reverts commit e346560, reversing
changes made to 5e0e168.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants