Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs(managers): clarify multi-line mode is not supported in regex custom manager #34571

Merged
merged 4 commits into from
Mar 5, 2025

Conversation

risu729
Copy link
Contributor

@risu729 risu729 commented Mar 2, 2025

Changes

Clarify the m flag, multi-line mode, of re2 is not supported in regex custom manager.

Context

The improvement is suggested here. #30170

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified unit tests, or
  • No unit tests but ran on a real repository, or
  • Both unit tests + ran on a real repository

@viceice viceice requested a review from HonkingGoose March 2, 2025 10:32
rarkins
rarkins previously approved these changes Mar 2, 2025
Co-authored-by: HonkingGoose <34918129+HonkingGoose@users.noreply.github.com>
risu729 and others added 2 commits March 5, 2025 19:12
Co-authored-by: HonkingGoose <34918129+HonkingGoose@users.noreply.github.com>
Co-authored-by: HonkingGoose <34918129+HonkingGoose@users.noreply.github.com>
@risu729
Copy link
Contributor Author

risu729 commented Mar 5, 2025

Thanks for the suggestions!

@HonkingGoose
Copy link
Collaborator

HonkingGoose commented Mar 5, 2025

Thank you for applying my suggestions!

Next time please put all suggestions into a batch, and then apply all the suggestions as a single commit. Read the GitHub Docs, applying suggested changes to learn how.

@risu729
Copy link
Contributor Author

risu729 commented Mar 5, 2025

My apologies for that. Thanks so much for the docs link.

@rarkins rarkins added this pull request to the merge queue Mar 5, 2025
Merged via the queue into renovatebot:main with commit 83ec919 Mar 5, 2025
39 checks passed
@renovate-release
Copy link
Collaborator

🎉 This PR is included in version 39.188.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

ivankatliarchuk added a commit to ivankatliarchuk/renovate that referenced this pull request Mar 7, 2025
* main: (84 commits)
  chore(deps): update dependency pdm to v2.22.4 (main) (renovatebot#34672)
  fix(helmv3): non http alias in Helm (renovatebot#34634)
  fix: improve coverage (renovatebot#34667)
  feat: new manager pixi (renovatebot#34400)
  feat(cache): Make http cache providers respect `cache-control` header (renovatebot#33848)
  chore(deps): update docker/dockerfile:1.14.0 docker digest to 4c68376 (main) (renovatebot#34660)
  chore(deps): update dependency @vitest/eslint-plugin to v1.1.36 (main) (renovatebot#34659)
  fix(gomod): read contraints from new `go.mod` (renovatebot#34655)
  fix: optimize type checks (renovatebot#34654)
  fix(manager/composer): use `intersects()` instead of `matches()` to compare tool constraint (renovatebot#34256)
  chore: fix Debugger launchers (renovatebot#34647)
  fix(devbox): use new --no-install flag to prevent devbox from using the nix store (renovatebot#34257)
  chore(deps): update dependency pnpm to v10.5.2 (main) (renovatebot#34648)
  chore(deps): update dependency @vitest/eslint-plugin to v1.1.35 (main) (renovatebot#34644)
  docs(managers): clarify multi-line mode is not supported in regex custom manager (renovatebot#34571)
  chore(deps): update dependency @vitest/eslint-plugin to v1.1.34 (main) (renovatebot#34639)
  fix(global): import config file as url (renovatebot#34637)
  feat: add RENOVATE_ prefix to Github com token and env host rules (renovatebot#34568)
  chore(deps): update dependency pnpm to v10.5.1 (main) (renovatebot#34631)
  feat(config): support `.mjs` config files (renovatebot#34626)
  ...
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.

4 participants