Skip to content

Conversation

@neha-p6
Copy link
Collaborator

@neha-p6 neha-p6 commented Nov 12, 2025

  • Remove chef gem form Gemfile, update comments, fix spec.
  • Remove license_scout as direct dependency from omnibus/Gemfile since omnibus also pulls in license_scout as its direct dependency, bundle update license_scout gets blocked. Bump omnibus to pull in latest license_scout to get pass licensing errors in adhoc build

Description

[Please describe what this change achieves]

Issues Resolved

[List any existing issues this PR resolves, or any Discourse or
StackOverflow discussions that are relevant]

Check List

@neha-p6 neha-p6 changed the title add back ruby platform in gemfile.lock Adhoc build fixes Nov 12, 2025
@neha-p6 neha-p6 force-pushed the neha-p6/jfm/chef19-remove-activesupport-take-2 branch from daa5d51 to 5f2fd0c Compare November 12, 2025 11:13
Remove chef gem form Gemfile, update comments, fix spec

Signed-off-by: neha-p6 <[email protected]>
@neha-p6 neha-p6 force-pushed the neha-p6/jfm/chef19-remove-activesupport-take-2 branch from 5f2fd0c to 9718483 Compare November 13, 2025 11:15
…omnibus also pulls in license_scout as its direct dependency, bundle update license_scout gets blocked. Bump omnibus to pull in latest license_scout

Signed-off-by: neha-p6 <[email protected]>

Lazy load evalution to avoid Nomethod Error

Signed-off-by: neha-p6 <[email protected]>
@neha-p6 neha-p6 force-pushed the neha-p6/jfm/chef19-remove-activesupport-take-2 branch from 9718483 to 46c4328 Compare November 13, 2025 13:40
Signed-off-by: neha-p6 <[email protected]>
@sonarqubecloud
Copy link

@neha-p6 neha-p6 marked this pull request as ready for review November 17, 2025 13:43
@neha-p6 neha-p6 requested review from a team as code owners November 17, 2025 13:43
@neha-p6 neha-p6 merged commit 7180875 into jfm/chef19-remove-activesupport-take-2 Nov 17, 2025
7 checks passed
@neha-p6 neha-p6 deleted the neha-p6/jfm/chef19-remove-activesupport-take-2 branch November 17, 2025 13:43
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