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

For QA:

Documentation:

Copy link

sonarqubecloud bot commented Oct 6, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@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.

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.

8 participants