Skip to content

Add action to build releases#54

Open
cgarling wants to merge 23 commits into
apace7:mainfrom
cgarling:gh-actions
Open

Add action to build releases#54
cgarling wants to merge 23 commits into
apace7:mainfrom
cgarling:gh-actions

Conversation

@cgarling

Copy link
Copy Markdown
Contributor

Closes #8.

This adds a manually triggered action to follow the steps in create_database.sh (I wrote the steps into the action itself, it doesn't actually use create_database.sh) and stage the output files as a github release. When manually triggered, the action will ask for a tag identifer, release notes, and there is an additional toggle to disable creating a release (you can toggle this off to do a "dry run").

You will see I made some additional changes to facilitate running on CI.

  • Mismatch between filename and key of ngc_0185_gc_pa-n185 was breaking build, fixed
  • path specifiers in some of the python files were brittle with respect to whether LVDBDIR environment variable had an ending '/' or not, used pathlib to harden this
  • Missing publishers for books in the lvdb.bib were causing issues so I added those (maybe this is not actually a fatal error, but I was troubleshooting failures and thought this would fix it; left it in anyway)
  • Removed duplicate entries in lvdb.bib (I think this was actually what was causing the builds to fail, not the missing publishers, but oh well)

See successful build run here:
https://github.com/cgarling/local_volume_database/actions/runs/25891102793/job/76094209244

which created this release: https://github.com/cgarling/local_volume_database/releases/tag/test1

You can peruse the files it created and see if anything is missing or looks incorrect. Let me know what you think, happy to iterate on this

Copilot AI and others added 23 commits May 14, 2026 15:17
Updated GitHub Actions workflow to include new inputs for release tag and dry run option, and upgraded action versions.
Updated the build_release workflow to check for existing tags and install dependencies from a requirements file.
Added 'cm-super' package to LaTeX installation.
Add texlive-publishers for revtex4-1
@apace7

apace7 commented May 15, 2026

Copy link
Copy Markdown
Owner

This is really cool to see. This is going to take me a bit to work through.

@cgarling cgarling mentioned this pull request May 15, 2026
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.

github actions

3 participants