Skip to content

Use chef/envy, not manderson26/envy#112

Open
lbakerchef wants to merge 1 commit intomainfrom
lbaker/envy
Open

Use chef/envy, not manderson26/envy#112
lbakerchef wants to merge 1 commit intomainfrom
lbaker/envy

Conversation

@lbakerchef
Copy link
Copy Markdown
Contributor

@lbakerchef lbakerchef commented Mar 23, 2021

Description

Use newly-forked chef/envy, not manderson26/envy.

Issue

chef/chef-server#2388

Related PRs

chef/chef-server#2289

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

  • I have read the CONTRIBUTING document.
  • I have run the pre-merge tests locally and they pass.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • All commits have been signed-off for the Developer Certificate of Origin.

@lbakerchef lbakerchef changed the title Standardize usage of deps (use markan/envy, not manderson26/envy) Use chef/envy, not manderson26/envy Apr 2, 2021
Signed-off-by: Lincoln Baker <lbaker@chef.io>
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented May 3, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information


{envy, ".*",
{git, "git://github.com/manderson26/envy.git", {branch, "master"}}}
{git, "git://github.com/chef/envy.git", {branch, "master"}}}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

would we also need to update the lock file?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants