Skip to content

add Stdlib::Syslogfacility type#1005

Merged
sheenaajay merged 1 commit intopuppetlabs:masterfrom
bastelfreak:facility
Sep 18, 2019
Merged

add Stdlib::Syslogfacility type#1005
sheenaajay merged 1 commit intopuppetlabs:masterfrom
bastelfreak:facility

Conversation

@bastelfreak
Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread types/syslogfacility.pp
bastelfreak added a commit to bastelfreak/puppet-nrpe that referenced this pull request Mar 9, 2019
The syslogfacility type is present in stdlib, so we should use it.
Before merging this,
puppetlabs/puppetlabs-stdlib#1005 needs to be
released. We also need to update metadata.json
Comment thread types/syslogfacility.pp Outdated
@sheenaajay
Copy link
Copy Markdown
Contributor

Hi @bastelfreak ,Hasn't seen any activity in this PR, do you think you'll be able to work on it in the near future? No pressure, just checking in.

@alexjfisher
Copy link
Copy Markdown
Collaborator

@sheenaajay Is there any extra work required? I don't think my nitpick should be a blocker.

@sheenaajay
Copy link
Copy Markdown
Contributor

Thank you @alexjfisher .
Hi @bastelfreak , Could you please resolve the conflicts and rebase with the latest master to run the acceptance tests.Thank you.

@bastelfreak
Copy link
Copy Markdown
Collaborator Author

@sheenaajay I rebased it

@sheenaajay
Copy link
Copy Markdown
Contributor

Screen Shot 2019-09-18 at 09 26 16

@sheenaajay
Copy link
Copy Markdown
Contributor

Thank you for submitting the PR @bastelfreak.

@sheenaajay
Copy link
Copy Markdown
Contributor

Great work...

@sheenaajay sheenaajay merged commit 645b826 into puppetlabs:master Sep 18, 2019
@bastelfreak bastelfreak deleted the facility branch September 19, 2019 21:03
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