Skip to content

allow tests/verify_mdb_properties.sh to be run in Docker #9176#12431

Open
pdurbin wants to merge 1 commit into
developfrom
9176-verify-props-docker
Open

allow tests/verify_mdb_properties.sh to be run in Docker #9176#12431
pdurbin wants to merge 1 commit into
developfrom
9176-verify-props-docker

Conversation

@pdurbin
Copy link
Copy Markdown
Member

@pdurbin pdurbin commented Jun 3, 2026

What this PR does / why we need it:

A problem with a properties file was reported here: https://github.com/IQSS/dataverse/actions/runs/26900153259/job/79350005084?pr=12425

I tried to run the script locally on my Mac but I got this error:

% tests/verify_mdb_properties.sh              
Cannot find jbang on path. Did you install it?

Let's run in it Docker! 🐳

After using the solution in this PR, I made this fix: 603b87f

Which issue(s) this PR closes:

None. The script was originally added here:

Special notes for your reviewer:

Changes were made by Codex.

Suggestions on how to test this:

See the README I updated (well, that Codex updated).

@pdurbin pdurbin moved this to Ready for Triage in IQSS Dataverse Project Jun 3, 2026
@pdurbin pdurbin requested a review from poikilotherm June 3, 2026 20:45
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 3, 2026

Test Results

397 tests  ±0   382 ✅ ±0   32m 42s ⏱️ + 7m 31s
 53 suites ±0    15 💤 ±0 
 53 files   ±0     0 ❌ ±0 

Results for commit 168c117. ± Comparison against base commit 0a1817f.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Ready for Triage

Development

Successfully merging this pull request may close these issues.

1 participant