Skip to content

Move syslog dependency from Gemfile to gemspec#298

Merged
nikhil2611 merged 7 commits into
mainfrom
nikhil/add-syslog-in-gemspec
Sep 11, 2025
Merged

Move syslog dependency from Gemfile to gemspec#298
nikhil2611 merged 7 commits into
mainfrom
nikhil/add-syslog-in-gemspec

Conversation

@nikhil2611

Copy link
Copy Markdown
Contributor

Description

In Ruby 3.4, syslog is no longer part of the stdlib and must be declared explicitly. Moved syslog from the Gemfile to the gemspec so it is always installed as a runtime dependency

Related Issue

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.
  • If Gemfile.lock has changed, I have used --conservative to do it and included the full output in the Description above.
  • All new and existing tests passed.
  • All commits have been signed-off for the Developer Certificate of Origin.

Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@nikhil2611 nikhil2611 requested review from a team as code owners September 9, 2025 08:39
@github-actions

github-actions Bot commented Sep 9, 2025

Copy link
Copy Markdown

Simplecov Report

Covered Threshold
98.51% 90%

nikhil2611 and others added 6 commits September 9, 2025 23:41
Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
Signed-off-by: Ashique Saidalavi <Ashique.saidalavi@progress.com>
Signed-off-by: Ashique Saidalavi <Ashique.saidalavi@progress.com>
Signed-off-by: Ashique Saidalavi <Ashique.saidalavi@progress.com>
Signed-off-by: Ashique Saidalavi <Ashique.saidalavi@progress.com>
Signed-off-by: nikhil2611 <nikhilgupta2102@gmail.com>
@sonarqubecloud

Copy link
Copy Markdown

@sonarqube-for-infrastructure-prod

Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@nikhil2611 nikhil2611 merged commit 52a026f into main Sep 11, 2025
12 checks passed
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.

3 participants