Skip to content

Drop concat_basedir fact in tests #1203

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

Merged
merged 1 commit into from
Nov 9, 2020

Conversation

ekohl
Copy link
Collaborator

@ekohl ekohl commented Nov 4, 2020

This fact hasn't existed since puppetlabs-concat 2.x as released in 2016. 6d827ec dropped support for those 1.x versions which means this hasn't been needed for a while.

@ekohl ekohl requested a review from a team as a code owner November 4, 2020 18:10
This fact hasn't existed since puppetlabs-concat 2.x as released in
2016. 6d827ec dropped support for those
1.x versions which means this hasn't been needed for a while.
@ekohl ekohl force-pushed the drop-concat-basedir branch from 673352e to 3fba56b Compare November 6, 2020 12:40
@codecov-io
Copy link

codecov-io commented Nov 6, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1203   +/-   ##
=======================================
  Coverage        ?   65.81%           
=======================================
  Files           ?       14           
  Lines           ?      351           
  Branches        ?        0           
=======================================
  Hits            ?      231           
  Misses          ?      120           
  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 2da32ae...3fba56b. Read the comment docs.

@sheenaajay sheenaajay self-assigned this Nov 9, 2020
@sheenaajay
Copy link
Contributor

Thanks alot @ekohl for your contribution.Changes looks good.

@sheenaajay sheenaajay merged commit 099ed05 into puppetlabs:main Nov 9, 2020
@ekohl ekohl deleted the drop-concat-basedir branch November 9, 2020 09:48
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.

3 participants