Skip to content

[WIP] Fix warnings raised by URI 1.0.1 #141

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jrafanie
Copy link
Member

@jrafanie jrafanie commented Nov 8, 2024

@jrafanie jrafanie force-pushed the fix-uri-1-0-1-warnings branch from 17fcced to d9b4b02 Compare November 8, 2024 17:58
agrare
agrare previously approved these changes Nov 8, 2024
@agrare agrare dismissed their stale review November 8, 2024 18:03

D'oh just saw the tests are failing

@jrafanie jrafanie force-pushed the fix-uri-1-0-1-warnings branch from d9b4b02 to 2da6fd7 Compare November 8, 2024 18:07
@jrafanie jrafanie changed the title Fix warnings raised by URI 1.0.1 [WIP] Fix warnings raised by URI 1.0.1 Nov 8, 2024
@jrafanie jrafanie force-pushed the fix-uri-1-0-1-warnings branch from 2da6fd7 to c649999 Compare November 8, 2024 21:29
@jrafanie jrafanie added the wip label Nov 11, 2024
@jrafanie jrafanie closed this Dec 10, 2024
@jrafanie jrafanie reopened this Dec 10, 2024
@miq-bot
Copy link
Member

miq-bot commented Mar 5, 2025

Checked commits jrafanie/vmware_web_service@c746723~...c649999 with ruby 3.1.5, rubocop 1.56.3, haml-lint 0.51.0, and yamllint
1 file checked, 0 offenses detected
Everything looks fine. 🍰

@miq-bot miq-bot added the stale label Jun 9, 2025
@miq-bot
Copy link
Member

miq-bot commented Jun 9, 2025

This pull request has been automatically marked as stale because it has not been updated for at least 3 months.

If these changes are still valid, please remove the stale label, make any changes requested by reviewers (if any), and ensure that this issue is being looked at by the assigned/reviewer(s).

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

Successfully merging this pull request may close these issues.

3 participants