Skip to content

Conversation

@ekjotsinghmakhija
Copy link
Member

This pull request introduces significant updates to workflows, dependencies, and code documentation. Key changes include restructuring GitHub Actions workflows for production and staging environments, upgrading dependencies in the Dockerfile, and improving code readability by adding detailed documentation to PostgreSQL decoder structs.

Workflow Updates

  • Production Workflow Changes:

    • Renamed .github/workflows/prod.yml to focus solely on deployment and removed release-related jobs. Deployment now triggers on merged pull requests to the releases/latest branch.
    • Introduced a new .github/workflows/prod_build.yml file to handle release builds, including multi-architecture Docker image creation and manifest merging.
    • Added .github/workflows/version_updated.yml to enforce version updates in mix.exs for pull requests.
  • Staging Workflow Changes:

    • Updated .github/workflows/staging.yml and .github/workflows/staging_linter.yml to align with naming conventions and adjusted database migration paths in the linter workflow. [1] [2]

Dependency Upgrades

  • Updated Dockerfile to use newer versions of Elixir (1.14.3), OTP (25.3), and Node.js (18.x), ensuring compatibility with modern tools and libraries. [1] [2]

Code Documentation Improvements

  • Enhanced the lib/extensions/postgres/adapters/postgres/decoder/decoder.ex file by adding detailed @moduledoc and @doc annotations for all decoder structs, improving code maintainability and developer understanding.
  • Added a module-level @moduledoc to oid_database.ex to clarify its purpose of mapping PostgreSQL type IDs.

Other Changes

  • Updated .releaserc to switch the release branch from release to main, aligning with the new branching strategy.
  • Added persistent volume mounts in Fly.io deployment configurations for both production and staging environments. [1] [2]

@ekjotsinghmakhija ekjotsinghmakhija merged commit 45468d1 into tealbase:main Jul 21, 2025
0 of 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