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

(IAC-1008) - Removal of Inappropriate Terminology #148

Conversation

david22swan
Copy link
Member

Removal of any offensive or inappropritate terminology from the code base

@puppet-community-rangefinder
Copy link

satellite_pe_tools is a class

that may have no external impact to Forge modules.

This module is declared in 0 of 576 indexed public Puppetfiles.


These results were generated with Rangefinder, a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report.

Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only.

@codecov-io
Copy link

codecov-io commented Feb 4, 2021

Codecov Report

❗ No coverage uploaded for pull request base (main@edf25e0). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##             main    #148   +/-   ##
======================================
  Coverage        ?   0.00%           
======================================
  Files           ?       5           
  Lines           ?     206           
  Branches        ?       0           
======================================
  Hits            ?       0           
  Misses          ?     206           
  Partials        ?       0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update edf25e0...1f2177c. Read the comment docs.

REFERENCE.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
config/roles.yaml Outdated Show resolved Hide resolved
manifests/init.pp Outdated Show resolved Hide resolved
@david22swan david22swan force-pushed the IAC-1008/main/removal_of_inappropriate_terminology branch from a3fa6ba to acca54c Compare February 4, 2021 14:18
@david22swan
Copy link
Member Author

david22swan commented Feb 4, 2021

@ekohl Thanks for your comments :)
Think I've resolved them all

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
manifests/init.pp Outdated Show resolved Hide resolved
@david22swan david22swan force-pushed the IAC-1008/main/removal_of_inappropriate_terminology branch from acca54c to 18f3873 Compare February 4, 2021 15:11
@david22swan
Copy link
Member Author

@ekohl Thank you for the updated comments, have made the suggested changes.

limp-utterance.delivery.puppetlabs.net-puppet-client.crt Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
@david22swan david22swan force-pushed the IAC-1008/main/removal_of_inappropriate_terminology branch 3 times, most recently from d0c8ada to d1ac4df Compare February 4, 2021 16:46
Comment on lines +9 to +10
# ssl_cert => '/etc/puppetlabs/puppet/ssl/certs/satellite-server.example.domain.pem',
# ssl_key => '/etc/puppetlabs/puppet/ssl/private_keys/puppet.domain.com.pem',
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking at

# @param ssl_cert The file path to the certificate signed by the Satellite CA. It's used for Satellite to verify the identity
# of the Puppet Enterprise master
# @param ssl_key The file path to the key for the Puppet Enterprise master generated by Satellite
it does suggest that this is a pair. However, this commit isn't very clear in why:

9446834

The fact we can't figure it out with documentation is not a good sign for this module.

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@david22swan david22swan force-pushed the IAC-1008/main/removal_of_inappropriate_terminology branch 3 times, most recently from 6982b4e to 5de399e Compare February 10, 2021 15:16
David Swan added 2 commits February 10, 2021 15:47
Removal of any offensive or inappropriate terminology from the code base
Initial pass, non-functional changes only.
Removal of any inappropriate terminology from the code base, functional changes included.
@david22swan david22swan force-pushed the IAC-1008/main/removal_of_inappropriate_terminology branch from 5de399e to 1f2177c Compare February 10, 2021 15:47
Copy link
Contributor

@michaeltlombardi michaeltlombardi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@michaeltlombardi michaeltlombardi merged commit ff2331f into puppetlabs:main Feb 10, 2021
@david22swan david22swan deleted the IAC-1008/main/removal_of_inappropriate_terminology branch April 6, 2022 11:19
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