Skip to content

Separate issue linking workflows, make them run on edited#14121

Merged
calixtus merged 2 commits into
JabRef:mainfrom
subhramit:subhramit-actions-1
Oct 20, 2025
Merged

Separate issue linking workflows, make them run on edited#14121
calixtus merged 2 commits into
JabRef:mainfrom
subhramit:subhramit-actions-1

Conversation

@subhramit

@subhramit subhramit commented Oct 20, 2025

Copy link
Copy Markdown
Member

Triggered by #14106 (comment)

Steps to test

NA

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.

@subhramit subhramit requested a review from koppor October 20, 2025 13:11
@subhramit subhramit mentioned this pull request Oct 20, 2025
Merged
6 tasks
@jabref-machine

jabref-machine commented Oct 20, 2025

Copy link
Copy Markdown
Collaborator

Your pull request needs to link an issue correctly.

To ease organizational workflows, please link this pull request to the issue by including a supported keyword in the pull request's description as per syntax described in GitHub's documentation.

Examples

  • ✅ Fixes #xyz links pull-request to issue. Merging the PR will close the issue.
  • ✅ Fixes https://github.com/JabRef/jabref/issues/xyz links pull-request to issue. Merging the PR will close the issue.
  • ✅ Fixes https://github.com/Koppor/jabref/issues/xyz links pull-request to issue. Merging the PR will close the issue.
  • ❌ Fixes [#xyz](https://github.com/JabRef/jabref/issues/xyz) links pull-request to issue. Merging the PR will NOT close the issue.

@Siedlerchr

Siedlerchr commented Oct 20, 2025

Copy link
Copy Markdown
Member

you should now link to an issue to test if the workflow works

@subhramit

subhramit commented Oct 20, 2025

Copy link
Copy Markdown
Member Author

you should now link to an issue to test if the workflow works

if I remember correctly, these run from the main branch of the base. So can test after merge.

@calixtus calixtus added this pull request to the merge queue Oct 20, 2025
Merged via the queue into JabRef:main with commit 016ea53 Oct 20, 2025
47 of 48 checks passed
@calixtus calixtus deleted the subhramit-actions-1 branch October 20, 2025 19:37
@subhramit

subhramit commented Oct 20, 2025

Copy link
Copy Markdown
Member Author

Tested in #14106
Works now.

Siedlerchr added a commit to Yubo-Cao/jabref that referenced this pull request Oct 20, 2025
@Siedlerchr
Merge remote-tracking branch 'upstream/main' into fix/786
3702c81 
* upstream/main:
  New study.yml format (JabRef#13844)
  Separate issue linking workflows, make them run on `edited` (JabRef#14121)
merlinymy pushed a commit to merlinymy/jabref that referenced this pull request Nov 19, 2025
@subhramit @merlinymy
Separate issue linking workflows, make them run on edited (JabRef#1…
9b2c5f6 
…4121)

* Separate issue linking workflows, make them run on `edited`

* Add workflow name to `workflow_run` for pr-comment
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

@koppor koppor Awaiting requested review from koppor

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@subhramit @jabref-machine @Siedlerchr @calixtus