Skip to content

ci(test): simplify db setup for github action#2365

Merged
PiTrem merged 1 commit intomainfrom
ci_update_db_setup
Mar 6, 2025
Merged

ci(test): simplify db setup for github action#2365
PiTrem merged 1 commit intomainfrom
ci_update_db_setup

Conversation

@PiTrem
Copy link
Member

@PiTrem PiTrem commented Mar 5, 2025

  • use superuser postgres as role for setting up the test db
    so that 'create extension' can be run without permission issue

  • rm unused template database.yml

@PiTrem PiTrem force-pushed the ci_update_db_setup branch from 7ff02bd to 14952ce Compare March 5, 2025 15:41
).first
puts "Warning: PG cartridge #{cartridge} version #{cartridge_version} is not installed" unless cartridge_present
end
rescue PG::ConnectionBad, ActiveRecord::NoDatabaseError => e
Copy link

Choose a reason for hiding this comment

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

Lint/SuppressedException: Do not suppress exceptions.

@PiTrem PiTrem mentioned this pull request Mar 5, 2025
@PiTrem PiTrem force-pushed the ci_update_db_setup branch 3 times, most recently from 5901ba5 to 27f3b4a Compare March 6, 2025 01:36
@PiTrem PiTrem force-pushed the ci_update_db_setup branch from 27f3b4a to 3040607 Compare March 6, 2025 01:57
@github-actions
Copy link

github-actions bot commented Mar 6, 2025

LCOV of commit 3040607 during Continuous Integration #4587

Summary coverage rate:
  lines......: 67.0% (15490 of 23120 lines)
  functions..: no data found
  branches...: no data found

Files changed coverage rate: n/a

@PiTrem PiTrem marked this pull request as draft March 6, 2025 04:16
@PiTrem PiTrem marked this pull request as ready for review March 6, 2025 04:28
@PiTrem PiTrem merged commit e1173a6 into main Mar 6, 2025
4 checks passed
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.

1 participant