+
Skip to content

Conversation

micszo
Copy link
Contributor

@micszo micszo commented Oct 2, 2025

🎫 Issue IBX-10494

Related PRs:

Description:

Added second "PostgreSQL integration tests" job to setup.
Before Postgres 14 was included, now Postgres 14 and 18 are included.

Additionally inconsistent BaseRepositoryFiltering test was fixed (f925665).

TODO:

  • matrix approach in integration tests
  • use of dedicated action for installation of composer packages
  • resolve unstable 'IsUserBased=true' test

For QA:

Documentation:

@micszo micszo marked this pull request as ready for review October 7, 2025 14:14
@micszo micszo requested review from a team, Steveb-p and alongosz October 7, 2025 14:28
@ezrobot ezrobot requested review from KamilSznajdrowicz, barbaragr, juskora, katarzynazawada, pawlakadrian and tomaszszopinski and removed request for a team October 7, 2025 14:28
- '8.1'

steps:
- uses: actions/checkout@v2

Choose a reason for hiding this comment

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

- uses: actions/create-github-app-token@v2 id: generate_token with: app-id: ${{ secrets.AUTOMATION_CLIENT_ID }} private-key: ${{ secrets.AUTOMATION_CLIENT_SECRET }} owner: ${{ github.repository_owner }}

is not needed here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Initially there was de8f501 but it left the PR after rebase. Other fixes to setup were not needed.

Copy link

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.

8 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载