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

(RE-12896) use artifactory instead of saturn/pe-releases #245

Merged
merged 1 commit into from
Feb 21, 2020

Conversation

sarameisburger
Copy link
Contributor

No description provided.

@codecov-io
Copy link

Codecov Report

Merging #245 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #245   +/-   ##
=======================================
  Coverage   72.83%   72.83%           
=======================================
  Files          17       17           
  Lines         773      773           
=======================================
  Hits          563      563           
  Misses        210      210

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 69ab182...340f65e. Read the comment docs.

Copy link
Contributor

@underscorgan underscorgan left a comment

Choose a reason for hiding this comment

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

lgtm, but will leave for modules team to merge

Copy link
Contributor

@genebean genebean left a comment

Choose a reason for hiding this comment

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

This is safe to merge as is since the associated beaker tests are not actually being used at this time.

@genebean genebean merged commit 2d8e385 into puppetlabs:master Feb 21, 2020
cegeka-jenkins pushed a commit to cegeka/puppet-lvm that referenced this pull request Jun 4, 2020
(RE-12896) use artifactory instead of saturn/pe-releases
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.

5 participants