Skip to content
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

Is gitium still considered suitable for production? #190

Closed
XuluWarrior opened this issue Jan 21, 2023 · 5 comments
Closed

Is gitium still considered suitable for production? #190

XuluWarrior opened this issue Jan 21, 2023 · 5 comments

Comments

@XuluWarrior
Copy link

There hasn't been a code update for three years. Which either says the code is rock solid or it's out of date.

@bogdanpetrea
Copy link
Contributor

Hi @XuluWarrior! We are still using it in production, and for our use case it does the job... so probably the former more than the latter.

@ibes
Copy link

ibes commented Mar 8, 2023

Just getting started with Gitium.
This seems to be an amazing tool for the WordPress eco system.
I don't understand, that it's not widely known and use.

Seems to need some work to be compatible with PHP8+

And it seems like it is not getting love from Presslabs anymore?!

Any ideas why?

@cristianuibar
Copy link
Contributor

Hello @ibes ,

We're still maintaining the plugin as it's being actively used by all of our customers right now, all running on PHP8 as this is the minimum on our platform.

I'd say it's in a low-maintenance mode right now, and you are correct, no new features have been added lately, but if any critical bug is found, we'll for sure address it.

Please let me know if you see any function that is not php8 ready, and I'll fix it.

@ibes
Copy link

ibes commented Mar 9, 2023

@cristianuibar

That is delightful to hear!
It's an awesome tool and I am stunned that it's not widely known and used "everywhere" :)

I needed to change something like here:
#158 (using the commentend out $wordpress_loader worked for me. Not tried the fix in this pull request)

And I had to change this line
$response = array_map( create_function( '$b', 'return str_replace("origin/","",$b);' ), $response ); in git-wrapper.php

But I just see: The WordPress.org version is not up to date.
Here on Github this is already fixed.

Would you be so kind and refresh the Version on wordpress.org if possible?

Likely my pain points are not your paint points just because the version on Github is up to date but the WordPress.org version is one or two steps behind.

I want to make sure to express:
I thing this plugin is an awesome service to the WordPress community and I am delighted that it in deed is maintained.

@DenisFlorin
Copy link
Contributor

The fix is already in place for the create_function on wp.org.

I will close this for now and continue investigating the problem #158

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

No branches or pull requests

5 participants