Skip to content

Conversation

@YerrabothulaNavyaSree09

This PR adds missing documentation on how RegEx works in JabRef’s search and dynamic groups.

Based on maintainer suggestion in issue #9093:
“might be nice to put this into the RegEx documentation.”

Added:

  • Examples of Java-style regex usage
  • Tips for escaping special characters
  • Dynamic group regex usage

@github-actions
Copy link
Contributor

Hey @YerrabothulaNavyaSree09!

Thank you for contributing to JabRef! Your help is truly appreciated ❤️

We have automated checks in place, based on which you will soon get feedback if any of them are failing. In a while, maintainers will also review your contribution. Once that happens, you can go through their comments in the "Files changed" tab and act on them, or reply to the conversation if you have further inputs.

Please re-check our contribution guide in case of any other doubts related to our contribution workflow.

@jabref-machine
Copy link
Collaborator

You have removed the "Mandatory Checks" section from your pull request description. Please adhere to our pull request template.

@jabref-machine
Copy link
Collaborator

Note that your PR will not be reviewed/accepted until you have gone through the mandatory checks in the description and marked each of them them exactly in the format of [x] (done), [ ] (not done yet) or [/] (not applicable).

@jabref-machine
Copy link
Collaborator

You modified Markdown (*.md) files and did not meet JabRef's rules for consistently formatted Markdown files. To ensure consistent styling, we have markdown-lint in place. Markdown lint's rules help to keep our Markdown files consistent within this repository and consistent with the Markdown files outside here.

You can check the detailed error output by navigating to your pull request, selecting the tab "Checks", section "Source Code Tests" (on the left), subsection "Markdown".

@ThiloteE ThiloteE marked this pull request as draft November 26, 2025 11:58
@ThiloteE
Copy link
Member

issue #9093 has nothing to do with regex.

@ThiloteE
Copy link
Member

ThiloteE commented Nov 27, 2025

Closing, because we already offer documentation about RegEx at this location in the documentation. If you want to continue working on this topic, then open another PR at the appropriate repository. Please avoid duplication in the documentation. You also have not addressed the automated feedback. Please avoid opening useless PRs, if there is no intent to keep working on your PR. Maintainers have to spend time on that, which they otherwise could use more meaningfully.

@ThiloteE ThiloteE closed this Nov 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants