Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support changing SchemaSpy version #173

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

borcsokj
Copy link

Changes

  • Exclude SchemaSpy artifact (JAR file) from repository, download it on the fly,
  • Support changing SchemaSpy version using environment variable (no need to wire it into GitHub action sources but default value only),
  • Increased PostgreSQL JDBC version (sync to latest official SchemaSpy version),
  • No need to publish SchemaSpy Docker image manually (i.e. to GitHub Container Registry), it is created by the workflow.

Example

The following example demonstrates how SchemaSpy version is overriden (so generated Types tab for enumerations of PostgreSQL database: https://github.com/borcsokj/schemaspy/tree/TypesTab):

      - name: Run SchemaSpy
        uses: 'tj-actions/schemaspy@v2' # tested as borcsokj/run-schemaspy@v2
        with:
          host: postgres
          port: 5432
          database_name: sandbox
          database_type: pgsql
          user: postgres
          password: *****
          all: true
        env:
          SCHEMASPY_GROUP_ID: io.github.borcsokj
          SCHEMASPY_ARTIFACT_ID: schemaspy
          SCHEMASPY_VERSION: 7.0.0-pre1

…on the fly

* support changing SchemaSpy version using environment variable (no need to wire it into GitHub action sources but default value only)
* increased PostgreSQL JDBC version (sync to latest official SchemaSpy version)
* no need to publish SchemaSpy Docker image manually (i.e. to GitHub Container Registry), it is created by the workflow
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