Skip to content

Add smoke tests#14338

Merged
Siedlerchr merged 1 commit into
mainfrom
add-smoke-test
Nov 17, 2025
Merged

Add smoke tests#14338
Siedlerchr merged 1 commit into
mainfrom
add-smoke-test

Conversation

@koppor

@koppor koppor commented Nov 17, 2025

Copy link
Copy Markdown
Member

Triggered by #14305

Mandatory checks

  • I own the copyright of the code submitted and I license it under the MIT license
  • [/] I manually tested my changes in running JabRef (always required)
  • [/] I added JUnit tests for changes (if applicable)
  • [/] I added screenshots in the PR description (if change is visible to the user)
  • [/] I described the change in CHANGELOG.md in a way that is understandable for the average user (if change is visible to the user)
  • [/] I checked the user documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request updating file(s) in https://github.com/JabRef/user-documentation/tree/main/en.

Siedlerchr
Siedlerchr reviewed Nov 17, 2025
View reviewed changes
Comment thread .github/workflows/binaries.yml
- name: Smoke test JabGui
shell: bash
run: |
./gradlew :jabgui:run --args="--help"

@Siedlerchr Siedlerchr Nov 17, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

do we need x11 gui then?

@koppor koppor Nov 17, 2025

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

No - seems to work without -- which is good :)

grafik

@Siedlerchr Siedlerchr added this pull request to the merge queue Nov 17, 2025
Merged via the queue into main with commit 6ebda27 Nov 17, 2025
65 of 70 checks passed
@Siedlerchr Siedlerchr deleted the add-smoke-test branch November 17, 2025 12:33
Siedlerchr added a commit that referenced this pull request Nov 17, 2025
@Siedlerchr
Merge remote-tracking branch 'upstream/main' into jackson3
c3ab787 
* upstream/main:
  Add smoke tests (#14338)
  Clean up packages in JabKit (#14335)
  Output binaries workflow decisions to summary (#14333)
  New translations jabref_en.properties (Portuguese, Brazilian) (#14332)
  Check for disk space only once (#14331)
  Chore(deps): Bump org.glassfish.jersey.core:jersey-server from 3.1.11 to 4.0.0 in /versions (#14305)
merlinymy pushed a commit to merlinymy/jabref that referenced this pull request Nov 19, 2025
@koppor @merlinymy
Add smoke tests (JabRef#14338)
a40e948
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Siedlerchr Siedlerchr Siedlerchr approved these changes

Assignees

No one assigned

Labels

dev: ci-cd

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@koppor @Siedlerchr