-
-
Notifications
You must be signed in to change notification settings - Fork 40
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
Add http scheme in url #53
Conversation
@DarkGhostHunter Can you take a look and see if we can merge this? |
Fix operator
Fix operator
@DarkGhostHunter I fixed the 2 errors related to this MR. |
Codecov ReportPatch coverage is 📢 Thoughts on this report? Let us know!. |
@DarkGhostHunter Can you run the checks, everything should be fixed. |
Kudos, SonarCloud Quality Gate passed!
|
@DarkGhostHunter Can we merge this PR please? |
@DarkGhostHunter Would it be possible to merge this please? |
Is this package still maintained? |
Yes, but I'm hands full under contract. Once it ends, I will get back into reviewing all PR's across the org. |
Hey, any news on this? We are using the package and we need to start testing our tickets and we can't test until these PR's are merged. I am not trying to rush you or anything, just wondering if I need to fork it until you can merge these or not. |
Fork it. I’m still under contract since the project got delayed. Italo Baeza C.El 08-11-2023, a la(s) 15:31, asivaneswaran ***@***.***> escribió:
Hey, any news on this?
We are using the package and we need to start testing our tickets and we can't test until these PR's are merged.
I am not trying to rush you or anything, just wondering if I need to fork it until you can merge these or not.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Went my own way #65 |
#50