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

Update github PR link to use the new solr repo #797

Merged
merged 3 commits into from
Apr 26, 2022

Conversation

janhoy
Copy link
Contributor

@janhoy janhoy commented Apr 8, 2022

When we convert CHANGES.txt to Changes.html, we support adding href links to text like "Pull Request #123". I counted and we only use it like 2 times, and both of those were incorrect, so I removed them.

NB: Perhaps we should allow a pattern like this as well, so you can add PRs to CHANGES without a JIRA:

* #794 Fix a bug

I.e. in addition to the pattern ((?:(?:(?:github|gh)\s+)?pull\s+request\s*(?:\#?\s*)?|gh-)(\d+)) also support ^\*\s+\#\d{1..4}.

solr/CHANGES.txt Outdated Show resolved Hide resolved
Co-authored-by: Christine Poerschke <[email protected]>
@janhoy janhoy merged commit d5ab86e into apache:main Apr 26, 2022
@janhoy janhoy deleted the changes2html-git-link branch April 26, 2022 10:16
janhoy added a commit that referenced this pull request Apr 26, 2022
* Remove the two existing PR references (#120 and #11) as none of those were actually correct with the old github repo anyway.
Co-authored-by: Christine Poerschke <[email protected]>

(cherry picked from commit d5ab86e)
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