Skip to content

Change Oracle image#18

Merged
kenjis merged 1 commit intomainfrom
change-oracle-image
Oct 13, 2023
Merged

Change Oracle image#18
kenjis merged 1 commit intomainfrom
change-oracle-image

Conversation

@kenjis
Copy link
Member

@kenjis kenjis commented Oct 12, 2023

Description
Fixes codeigniter4/shield#899

See codeigniter4/CodeIgniter4#6755
See https://github.com/gvenzl/oci-oracle-xe#github-actions

Checklist:

  • Securely signed commits
  • [] Component(s) with PHPDoc blocks, only if necessary or adds value
  • [] Unit testing, with >80% coverage
  • [] User guide updated
  • [] Conforms to style guide

@kenjis kenjis force-pushed the change-oracle-image branch from 3473bfa to 06a45e8 Compare October 12, 2023 22:46
Copy link
Member

@MGatner MGatner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't understand this entirely but I see it works and trust your research.

@kenjis
Copy link
Member Author

kenjis commented Oct 13, 2023

The idea is to change to another Oracle image that Paul found.
In fact, the execution time is shorter because there is no Oracle client installation.

This is the result of this new Oracle image:
Screenshot 2023-10-13 19 01 21

This is the result of the current Oracle image:
Screenshot 2023-10-13 19 01 33

@kenjis kenjis merged commit f137f97 into main Oct 13, 2023
@kenjis kenjis deleted the change-oracle-image branch October 13, 2023 10:08
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.

Testing with OCI8 always fails

3 participants