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

(CONT-1048) - Removal of deprecated node_encrypt::file defined type #94

Merged
merged 2 commits into from
Jun 9, 2023

Conversation

jordanbreen28
Copy link
Contributor

@jordanbreen28 jordanbreen28 commented Jun 7, 2023

This PR removes the deprecated node_encrypt::file defined type. This type was retained for backwards code compatibility, but was just a wrapper for the node_encrypt::secret function.

This PR removes this deprecated defined type, in preparation of the upcoming major release.

@jordanbreen28 jordanbreen28 requested a review from a team as a code owner June 7, 2023 12:57
@jordanbreen28 jordanbreen28 changed the title (CONT-1048) - Removal of Deprecated node_encrypt::file (CONT-1048) - Removal of deprecated node_encrypt::file defined type Jun 7, 2023
jordanbreen28 added a commit that referenced this pull request Jun 8, 2023
Skipping this test due to a difference in behaviour between mocha stub
and rspec allow when calling the PuppetX::Binford2k::NodeEncrypt.decrypt
method on Ruby 2.x only.

Voted by the team to skip, as test case will be removed in
#94 with the
removal of the deprecated node_encrypt::file defined type.
jordanbreen28 added a commit that referenced this pull request Jun 8, 2023
Skipping this test due to a difference in behaviour between mocha stub
and rspec allow when calling the PuppetX::Binford2k::NodeEncrypt.decrypt
method on Ruby 2.x only.

Voted by the team to skip, as test case will be removed in
#94 with the
removal of the deprecated node_encrypt::file defined type.
@jordanbreen28 jordanbreen28 force-pushed the CONT-1048-remove_deprecated_file_type branch from bcbd308 to 2c0da98 Compare June 8, 2023 15:22
@jordanbreen28 jordanbreen28 marked this pull request as draft June 9, 2023 07:42
This commit removes the now deprecated node_encrypt::file defined type.
This type was retained for backwards code compatiblity, but was just a
wrapper for the deferred function.

This commit removes this deprecated defined type, in preparation of the
upcoming major release.
@jordanbreen28 jordanbreen28 force-pushed the CONT-1048-remove_deprecated_file_type branch from 2c0da98 to 1a83267 Compare June 9, 2023 08:45
@jordanbreen28 jordanbreen28 marked this pull request as ready for review June 9, 2023 08:53
@GSPatton GSPatton merged commit 50d19a4 into main Jun 9, 2023
@GSPatton GSPatton deleted the CONT-1048-remove_deprecated_file_type branch June 9, 2023 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants