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

Remove i386 Linux from push #11922

Closed
wants to merge 1 commit into from

Conversation

iluuu1994
Copy link
Member

Testing this in nightly is sufficient. 32-bit problems are rare.

Testing this in nightly is sufficient. 32-bit problems are rare.
@Girgias
Copy link
Member

Girgias commented Aug 9, 2023

I agree that most of the time 32 bit is not really an issue, but for tweaking core engine code it might be useful to still have the template and to be able to toggle it on when needed?

@iluuu1994
Copy link
Member Author

I looked into triggering pipelines on demand a while back through comments (for performance measuring) but couldn't find a good way to do it. One way to do it could be to modify the title/content and then manually retrigger the CI. To be honest though, I don't think it's necessary. I look at nightly, when there's an issue I can report it.

@Girgias
Copy link
Member

Girgias commented Aug 9, 2023

That's fair, worst case one can always readd the CI run for a specific PR.

@iluuu1994 iluuu1994 closed this in 248e6b0 Aug 10, 2023
ju1ius pushed a commit to ju1ius/php-src that referenced this pull request Aug 15, 2023
Testing this in nightly is sufficient. 32-bit problems are rare.

Closes phpGH-11922
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.

None yet

2 participants