Fix simplest compiler warnings in OslcClient pragmas

[berezovskyi]

Fix simplest compiler warnings in OslcClient pragmas

---

PR created automatically by Jules for task 8717113622136403359 started by @berezovskyi

Summary by CodeRabbit

• Chores
  • Internal compiler directive adjustments for code quality tooling.

Note: This release contains no user-facing changes or new functionality.

[google-labs-jules]

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.

---

For security, I will only act on instructions from the user who triggered this task.

[coderabbitai]

Warning

Rate limit exceeded

@berezovskyi has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 44 minutes and 30 seconds before requesting another review.

To keep reviews running without waiting, you can enable usage-based add-on for your organization. This allows additional reviews beyond the hourly cap. Account admins can enable it under billing.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 18f9675c-273f-429b-9b41-fd0f9927e020

📥 Commits

Reviewing files that changed from the base of the PR and between de3153a and cba6a59.

📒 Files selected for processing (1)

• OSLC4Net_SDK/OSLC4Net.Client/Oslc/OslcClient.cs

Walkthrough

The PR updates compiler pragma directives for code analysis rule MA0039 in OslcClient constructor methods. It replaces a warning enable directive with a restore directive in the obsolete cert-callback constructor and wraps the dangerous certificate validator assignment with disable/restore directives.

Changes

Compiler Pragma Configuration

Layer / File(s)	Summary
Pragma Warning Directives OSLC4Net_SDK/OSLC4Net.Client/Oslc/OslcClient.cs	Updated MA0039 rule suppression: replaced enable with restore in obsolete constructor; wrapped dangerous certificate validator assignment with disable/restore directives.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A pragma here, a pragma there,
Warning rules we gently tare,
MA0039 takes a bow,
Suppressed just right, we know just how—
Code analysis, tamed with flair!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically summarizes the main change: fixing compiler warnings in OslcClient pragmas by replacing enable with restore directives.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix-oslc-client-pragmas-8717113622136403359

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Review rate limit: 0/1 reviews remaining, refill in 44 minutes and 30 seconds.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 51.87%. Comparing base (970d9a0) to head (cba6a59).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #724   +/-   ##
=======================================
  Coverage   51.87%   51.87%           
=======================================
  Files         174      174           
  Lines       10248    10248           
  Branches     1020     1020           
=======================================
  Hits         5316     5316           
  Misses       4677     4677           
  Partials      255      255           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.