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

orphan fragments that contain a number in the filename are linked to issues. #562

Closed
mdellweg opened this issue Nov 6, 2023 · 3 comments · Fixed by #564
Closed

orphan fragments that contain a number in the filename are linked to issues. #562

mdellweg opened this issue Nov 6, 2023 · 3 comments · Fixed by #564

Comments

@mdellweg
Copy link
Contributor

mdellweg commented Nov 6, 2023

e.g. calling the fragment CHANGES/+compat.somelib_1.12.feature will link the supposed orphan snippet to the issue #12.

@adiroiban
Copy link
Member

adiroiban commented Nov 6, 2023

Thanks for the report

It looks like we have some missing automated tests.

I think that the fix can start by writing an automated test (or updating the current one) for this filename and see the test failing.

I think that the issue is somewhere here:

for basename in files:
ticket, category, counter = parse_newfragment_basename(
basename, frag_type_names
)
if category is None:
continue
assert ticket is not None
assert counter is not None
if orphan_prefix and ticket.startswith(orphan_prefix):
ticket = ""
# Use and increment the orphan news fragment counter.
counter = orphan_fragment_counter[category]
orphan_fragment_counter[category] += 1

or here

def parse_newfragment_basename(
basename: str, frag_type_names: Iterable[str]
) -> tuple[str, str, int] | tuple[None, None, None]:
invalid = (None, None, None)
parts = basename.split(".")
if len(parts) == 1:
return invalid
if len(parts) == 2:
ticket, category = parts
ticket = strip_if_integer_string(ticket)
return (ticket, category, 0) if category in frag_type_names else invalid
# There are at least 3 parts. Search for a valid category from the second
# part onwards.
# The category is used as the reference point in the parts list to later
# infer the issue number and counter value.
for i in range(1, len(parts)):
if parts[i] in frag_type_names:
# Current part is a valid category according to given definitions.
category = parts[i]
# Use the previous part as the ticket number.
# NOTE: This allows news fragment names like fix-1.2.3.feature or
# something-cool.feature.ext for projects that don't use ticket
# numbers in news fragment names.
ticket = strip_if_integer_string(parts[i - 1])
counter = 0
# Use the following part as the counter if it exists and is a valid
# digit.
if len(parts) > (i + 1) and parts[i + 1].isdigit():
counter = int(parts[i + 1])
return ticket, category, counter
else:
# No valid category found.
return invalid

I would be happy to review a fix for this :)

@mdellweg
Copy link
Contributor Author

mdellweg commented Nov 7, 2023

I started adding to a test: #564
Do you want to see it red before fixing it?

Also your two code snippets are the same. Did you mention to point me to another area too?

@adiroiban
Copy link
Member

Awesome. I have enabled the tests on that PR.

I have updated the code link

mdellweg added a commit to mdellweg/towncrier that referenced this issue Nov 7, 2023
mdellweg added a commit to mdellweg/towncrier that referenced this issue Nov 7, 2023
mdellweg added a commit to mdellweg/towncrier that referenced this issue Nov 7, 2023
mdellweg added a commit to mdellweg/towncrier that referenced this issue Nov 7, 2023
mdellweg added a commit to mdellweg/towncrier that referenced this issue Nov 7, 2023
fixes twisted#562

Co-authored-by: Adi Roiban <adiroiban@gmail.com>
adiroiban added a commit that referenced this issue Nov 7, 2023
* Add test for orphan fragments with numbers

* Fix fragment parsing with stray numbers and dots

fixes #562

Co-authored-by: Adi Roiban <adiroiban@gmail.com>
github-actions bot pushed a commit to aio-libs/aiohttp that referenced this issue Nov 9, 2023
Bumps [towncrier](https://github.com/twisted/towncrier) from 23.10.0 to
23.11.0.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a
href="https://github.com/twisted/towncrier/releases">towncrier's
releases</a>.</em></p>
<blockquote>
<h2>Towncrier 23.11.0</h2>
<h1>towncrier 23.11.0 (2023-11-08)</h1>
<h2>Bugfixes</h2>
<ul>
<li><code>build</code> now treats a missing fragments directory the same
as an
empty one, consistent with other operations.
(<a
href="https://redirect.github.com/twisted/towncrier/issues/538">#538</a>)</li>
<li>Fragments with filenames like
<!-- raw HTML omitted -->fix-1.2.3.feature<!-- raw HTML omitted --> are
now associated
with the ticket <!-- raw HTML omitted -->fix-1.2.3<!-- raw HTML omitted
-->. In
previous versions they were incorrectly associated to ticket
<!-- raw HTML omitted -->3<!-- raw HTML omitted -->.
(<a
href="https://redirect.github.com/twisted/towncrier/issues/562">#562</a>)</li>
<li>Orphan newsfragments containing numeric values are no longer
accidentally associated to tickets. In previous versions the orphan
marker was ignored and the newsfragment was associated to a ticket
having the last numerical value from the filename.
(<a
href="https://redirect.github.com/twisted/towncrier/issues/562">#562</a>)</li>
</ul>
<h2>Misc</h2>
<ul>
<li><a
href="https://redirect.github.com/twisted/towncrier/issues/558">#558</a>,
<a
href="https://redirect.github.com/twisted/towncrier/issues/559">#559</a></li>
</ul>
<h2>Towncrier 23.11.0rc1</h2>
<h1>towncrier 23.11.0rc1 (2023-11-07)</h1>
<h2>Bugfixes</h2>
<ul>
<li><code>build</code> now treats a missing fragments directory the same
as an
empty one, consistent with other operations.
(<a
href="https://redirect.github.com/twisted/towncrier/issues/538">#538</a>)</li>
<li>Fragments with filenames like
<!-- raw HTML omitted -->fix-1.2.3.feature<!-- raw HTML omitted --> are
now associated
with the ticket <!-- raw HTML omitted -->fix-1.2.3<!-- raw HTML omitted
-->. In
previous versions they were incorrectly associated to ticket
<!-- raw HTML omitted -->3<!-- raw HTML omitted -->.
(<a
href="https://redirect.github.com/twisted/towncrier/issues/562">#562</a>)</li>
<li>Orphan newsfragments containing numeric values are no longer
accidentally associated to tickets. In previous versions the orphan
marker was ignored and the newsfragment was associated to a ticket
having the last numerical value from the filename.
(<a
href="https://redirect.github.com/twisted/towncrier/issues/562">#562</a>)</li>
</ul>
<h2>Misc</h2>
<ul>
<li><a
href="https://redirect.github.com/twisted/towncrier/issues/558">#558</a>,
<a
href="https://redirect.github.com/twisted/towncrier/issues/559">#559</a></li>
</ul>
</blockquote>
</details>
<details>
<summary>Changelog</summary>
<p><em>Sourced from <a
href="https://github.com/twisted/towncrier/blob/trunk/NEWS.rst">towncrier's
changelog</a>.</em></p>
<blockquote>
<h1>towncrier 23.11.0 (2023-11-08)</h1>
<p>No significant changes since the previous release candidate.</p>
<h2>Bugfixes</h2>
<ul>
<li><code>build</code> now treats a missing fragments directory the same
as an empty one, consistent with other operations.
(<code>[#538](twisted/towncrier#538)
&lt;https://github.com/twisted/towncrier/issues/538&gt;</code>_)</li>
<li>Fragments with filenames like <code>fix-1.2.3.feature</code> are now
associated with the ticket <code>fix-1.2.3</code>.
In previous versions they were incorrectly associated to ticket
<code>3</code>.
(<code>[#562](twisted/towncrier#562)
&lt;https://github.com/twisted/towncrier/issues/562&gt;</code>_)</li>
<li>Orphan newsfragments containing numeric values are no longer
accidentally associated to tickets. In previous versions the orphan
marker was ignored and the newsfragment was associated to a ticket
having the last numerical value from the filename.
(<code>[#562](twisted/towncrier#562)
&lt;https://github.com/twisted/towncrier/issues/562&gt;</code>_)</li>
</ul>
<h2>Misc</h2>
<ul>
<li><code>[#558](twisted/towncrier#558)
&lt;https://github.com/twisted/towncrier/issues/558&gt;</code><em>,
<code>[#559](twisted/towncrier#559)
&lt;https://github.com/twisted/towncrier/issues/559&gt;</code></em></li>
</ul>
</blockquote>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a
href="https://github.com/twisted/towncrier/commit/503b499a1ddb13d6c526e5d5511767e53d809e68"><code>503b499</code></a>
Update final version.</li>
<li><a
href="https://github.com/twisted/towncrier/commit/bdb1d080e40babc7da54d3f6def37862d0cdcb94"><code>bdb1d08</code></a>
[pre-commit.ci] auto fixes from pre-commit.com hooks</li>
<li><a
href="https://github.com/twisted/towncrier/commit/47cf0f3b1d5f4be082fbddbad9b945274eb4d6bb"><code>47cf0f3</code></a>
Show release draft as a step output.</li>
<li><a
href="https://github.com/twisted/towncrier/commit/8b9b4b24136c628c874400ad1e9dd3a4a0a2c9bc"><code>8b9b4b2</code></a>
venv/bin/towncrier build --yes</li>
<li><a
href="https://github.com/twisted/towncrier/commit/3b27ce2e7ec2f219a8a655001bc21e5d4ba57412"><code>3b27ce2</code></a>
Fix orphan fragments with numbers (<a
href="https://redirect.github.com/twisted/towncrier/issues/564">#564</a>)</li>
<li><a
href="https://github.com/twisted/towncrier/commit/a275be976216b7e11b896b63df7fd9fcbccc0023"><code>a275be9</code></a>
[pre-commit.ci] pre-commit autoupdate (<a
href="https://redirect.github.com/twisted/towncrier/issues/563">#563</a>)</li>
<li><a
href="https://github.com/twisted/towncrier/commit/a794718b345cb332cd16ce9a807f4c02c68088d3"><code>a794718</code></a>
Update the documentation for the release process. (<a
href="https://redirect.github.com/twisted/towncrier/issues/558">#558</a>)</li>
<li><a
href="https://github.com/twisted/towncrier/commit/fba26a53022d6b5199a171018e0532a9ee59b596"><code>fba26a5</code></a>
Update build-and-inspect-python-package to 1.5.2 (<a
href="https://redirect.github.com/twisted/towncrier/issues/559">#559</a>)</li>
<li><a
href="https://github.com/twisted/towncrier/commit/e4b892fb21afce321a132780b7d4cf34814da7cb"><code>e4b892f</code></a>
Make consistent the expectation around non-existent fragments directory
(<a
href="https://redirect.github.com/twisted/towncrier/issues/557">#557</a>)</li>
<li><a
href="https://github.com/twisted/towncrier/commit/f6809f031fce9fe3568166247f8ccf8f7a6c4aaf"><code>f6809f0</code></a>
Release 23.10.0 (<a
href="https://redirect.github.com/twisted/towncrier/issues/555">#555</a>)</li>
<li>See full diff in <a
href="https://github.com/twisted/towncrier/compare/23.10.0...23.11.0">compare
view</a></li>
</ul>
</details>
<br />


[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=towncrier&package-manager=pip&previous-version=23.10.0&new-version=23.11.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after
your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge
and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating
it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)


</details>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
github-actions bot pushed a commit to aio-libs/aiohttp that referenced this issue Nov 9, 2023
Bumps [towncrier](https://github.com/twisted/towncrier) from 23.10.0 to
23.11.0.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a
href="https://github.com/twisted/towncrier/releases">towncrier's
releases</a>.</em></p>
<blockquote>
<h2>Towncrier 23.11.0</h2>
<h1>towncrier 23.11.0 (2023-11-08)</h1>
<h2>Bugfixes</h2>
<ul>
<li><code>build</code> now treats a missing fragments directory the same
as an
empty one, consistent with other operations.
(<a
href="https://redirect.github.com/twisted/towncrier/issues/538">#538</a>)</li>
<li>Fragments with filenames like
<!-- raw HTML omitted -->fix-1.2.3.feature<!-- raw HTML omitted --> are
now associated
with the ticket <!-- raw HTML omitted -->fix-1.2.3<!-- raw HTML omitted
-->. In
previous versions they were incorrectly associated to ticket
<!-- raw HTML omitted -->3<!-- raw HTML omitted -->.
(<a
href="https://redirect.github.com/twisted/towncrier/issues/562">#562</a>)</li>
<li>Orphan newsfragments containing numeric values are no longer
accidentally associated to tickets. In previous versions the orphan
marker was ignored and the newsfragment was associated to a ticket
having the last numerical value from the filename.
(<a
href="https://redirect.github.com/twisted/towncrier/issues/562">#562</a>)</li>
</ul>
<h2>Misc</h2>
<ul>
<li><a
href="https://redirect.github.com/twisted/towncrier/issues/558">#558</a>,
<a
href="https://redirect.github.com/twisted/towncrier/issues/559">#559</a></li>
</ul>
<h2>Towncrier 23.11.0rc1</h2>
<h1>towncrier 23.11.0rc1 (2023-11-07)</h1>
<h2>Bugfixes</h2>
<ul>
<li><code>build</code> now treats a missing fragments directory the same
as an
empty one, consistent with other operations.
(<a
href="https://redirect.github.com/twisted/towncrier/issues/538">#538</a>)</li>
<li>Fragments with filenames like
<!-- raw HTML omitted -->fix-1.2.3.feature<!-- raw HTML omitted --> are
now associated
with the ticket <!-- raw HTML omitted -->fix-1.2.3<!-- raw HTML omitted
-->. In
previous versions they were incorrectly associated to ticket
<!-- raw HTML omitted -->3<!-- raw HTML omitted -->.
(<a
href="https://redirect.github.com/twisted/towncrier/issues/562">#562</a>)</li>
<li>Orphan newsfragments containing numeric values are no longer
accidentally associated to tickets. In previous versions the orphan
marker was ignored and the newsfragment was associated to a ticket
having the last numerical value from the filename.
(<a
href="https://redirect.github.com/twisted/towncrier/issues/562">#562</a>)</li>
</ul>
<h2>Misc</h2>
<ul>
<li><a
href="https://redirect.github.com/twisted/towncrier/issues/558">#558</a>,
<a
href="https://redirect.github.com/twisted/towncrier/issues/559">#559</a></li>
</ul>
</blockquote>
</details>
<details>
<summary>Changelog</summary>
<p><em>Sourced from <a
href="https://github.com/twisted/towncrier/blob/trunk/NEWS.rst">towncrier's
changelog</a>.</em></p>
<blockquote>
<h1>towncrier 23.11.0 (2023-11-08)</h1>
<p>No significant changes since the previous release candidate.</p>
<h2>Bugfixes</h2>
<ul>
<li><code>build</code> now treats a missing fragments directory the same
as an empty one, consistent with other operations.
(<code>[#538](twisted/towncrier#538)
&lt;https://github.com/twisted/towncrier/issues/538&gt;</code>_)</li>
<li>Fragments with filenames like <code>fix-1.2.3.feature</code> are now
associated with the ticket <code>fix-1.2.3</code>.
In previous versions they were incorrectly associated to ticket
<code>3</code>.
(<code>[#562](twisted/towncrier#562)
&lt;https://github.com/twisted/towncrier/issues/562&gt;</code>_)</li>
<li>Orphan newsfragments containing numeric values are no longer
accidentally associated to tickets. In previous versions the orphan
marker was ignored and the newsfragment was associated to a ticket
having the last numerical value from the filename.
(<code>[#562](twisted/towncrier#562)
&lt;https://github.com/twisted/towncrier/issues/562&gt;</code>_)</li>
</ul>
<h2>Misc</h2>
<ul>
<li><code>[#558](twisted/towncrier#558)
&lt;https://github.com/twisted/towncrier/issues/558&gt;</code><em>,
<code>[#559](twisted/towncrier#559)
&lt;https://github.com/twisted/towncrier/issues/559&gt;</code></em></li>
</ul>
</blockquote>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a
href="https://github.com/twisted/towncrier/commit/503b499a1ddb13d6c526e5d5511767e53d809e68"><code>503b499</code></a>
Update final version.</li>
<li><a
href="https://github.com/twisted/towncrier/commit/bdb1d080e40babc7da54d3f6def37862d0cdcb94"><code>bdb1d08</code></a>
[pre-commit.ci] auto fixes from pre-commit.com hooks</li>
<li><a
href="https://github.com/twisted/towncrier/commit/47cf0f3b1d5f4be082fbddbad9b945274eb4d6bb"><code>47cf0f3</code></a>
Show release draft as a step output.</li>
<li><a
href="https://github.com/twisted/towncrier/commit/8b9b4b24136c628c874400ad1e9dd3a4a0a2c9bc"><code>8b9b4b2</code></a>
venv/bin/towncrier build --yes</li>
<li><a
href="https://github.com/twisted/towncrier/commit/3b27ce2e7ec2f219a8a655001bc21e5d4ba57412"><code>3b27ce2</code></a>
Fix orphan fragments with numbers (<a
href="https://redirect.github.com/twisted/towncrier/issues/564">#564</a>)</li>
<li><a
href="https://github.com/twisted/towncrier/commit/a275be976216b7e11b896b63df7fd9fcbccc0023"><code>a275be9</code></a>
[pre-commit.ci] pre-commit autoupdate (<a
href="https://redirect.github.com/twisted/towncrier/issues/563">#563</a>)</li>
<li><a
href="https://github.com/twisted/towncrier/commit/a794718b345cb332cd16ce9a807f4c02c68088d3"><code>a794718</code></a>
Update the documentation for the release process. (<a
href="https://redirect.github.com/twisted/towncrier/issues/558">#558</a>)</li>
<li><a
href="https://github.com/twisted/towncrier/commit/fba26a53022d6b5199a171018e0532a9ee59b596"><code>fba26a5</code></a>
Update build-and-inspect-python-package to 1.5.2 (<a
href="https://redirect.github.com/twisted/towncrier/issues/559">#559</a>)</li>
<li><a
href="https://github.com/twisted/towncrier/commit/e4b892fb21afce321a132780b7d4cf34814da7cb"><code>e4b892f</code></a>
Make consistent the expectation around non-existent fragments directory
(<a
href="https://redirect.github.com/twisted/towncrier/issues/557">#557</a>)</li>
<li><a
href="https://github.com/twisted/towncrier/commit/f6809f031fce9fe3568166247f8ccf8f7a6c4aaf"><code>f6809f0</code></a>
Release 23.10.0 (<a
href="https://redirect.github.com/twisted/towncrier/issues/555">#555</a>)</li>
<li>See full diff in <a
href="https://github.com/twisted/towncrier/compare/23.10.0...23.11.0">compare
view</a></li>
</ul>
</details>
<br />


[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=towncrier&package-manager=pip&previous-version=23.10.0&new-version=23.11.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after
your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge
and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating
it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)


</details>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
xiangxli pushed a commit to xiangxli/aiohttp that referenced this issue Dec 4, 2023
Bumps [towncrier](https://github.com/twisted/towncrier) from 23.10.0 to
23.11.0.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a
href="https://github.com/twisted/towncrier/releases">towncrier's
releases</a>.</em></p>
<blockquote>
<h2>Towncrier 23.11.0</h2>
<h1>towncrier 23.11.0 (2023-11-08)</h1>
<h2>Bugfixes</h2>
<ul>
<li><code>build</code> now treats a missing fragments directory the same
as an
empty one, consistent with other operations.
(<a
href="https://redirect.github.com/twisted/towncrier/issues/538">#538</a>)</li>
<li>Fragments with filenames like
<!-- raw HTML omitted -->fix-1.2.3.feature<!-- raw HTML omitted --> are
now associated
with the ticket <!-- raw HTML omitted -->fix-1.2.3<!-- raw HTML omitted
-->. In
previous versions they were incorrectly associated to ticket
<!-- raw HTML omitted -->3<!-- raw HTML omitted -->.
(<a
href="https://redirect.github.com/twisted/towncrier/issues/562">#562</a>)</li>
<li>Orphan newsfragments containing numeric values are no longer
accidentally associated to tickets. In previous versions the orphan
marker was ignored and the newsfragment was associated to a ticket
having the last numerical value from the filename.
(<a
href="https://redirect.github.com/twisted/towncrier/issues/562">#562</a>)</li>
</ul>
<h2>Misc</h2>
<ul>
<li><a
href="https://redirect.github.com/twisted/towncrier/issues/558">#558</a>,
<a
href="https://redirect.github.com/twisted/towncrier/issues/559">#559</a></li>
</ul>
<h2>Towncrier 23.11.0rc1</h2>
<h1>towncrier 23.11.0rc1 (2023-11-07)</h1>
<h2>Bugfixes</h2>
<ul>
<li><code>build</code> now treats a missing fragments directory the same
as an
empty one, consistent with other operations.
(<a
href="https://redirect.github.com/twisted/towncrier/issues/538">#538</a>)</li>
<li>Fragments with filenames like
<!-- raw HTML omitted -->fix-1.2.3.feature<!-- raw HTML omitted --> are
now associated
with the ticket <!-- raw HTML omitted -->fix-1.2.3<!-- raw HTML omitted
-->. In
previous versions they were incorrectly associated to ticket
<!-- raw HTML omitted -->3<!-- raw HTML omitted -->.
(<a
href="https://redirect.github.com/twisted/towncrier/issues/562">#562</a>)</li>
<li>Orphan newsfragments containing numeric values are no longer
accidentally associated to tickets. In previous versions the orphan
marker was ignored and the newsfragment was associated to a ticket
having the last numerical value from the filename.
(<a
href="https://redirect.github.com/twisted/towncrier/issues/562">#562</a>)</li>
</ul>
<h2>Misc</h2>
<ul>
<li><a
href="https://redirect.github.com/twisted/towncrier/issues/558">#558</a>,
<a
href="https://redirect.github.com/twisted/towncrier/issues/559">#559</a></li>
</ul>
</blockquote>
</details>
<details>
<summary>Changelog</summary>
<p><em>Sourced from <a
href="https://github.com/twisted/towncrier/blob/trunk/NEWS.rst">towncrier's
changelog</a>.</em></p>
<blockquote>
<h1>towncrier 23.11.0 (2023-11-08)</h1>
<p>No significant changes since the previous release candidate.</p>
<h2>Bugfixes</h2>
<ul>
<li><code>build</code> now treats a missing fragments directory the same
as an empty one, consistent with other operations.
(<code>[aio-libs#538](twisted/towncrier#538)
&lt;https://github.com/twisted/towncrier/issues/538&gt;</code>_)</li>
<li>Fragments with filenames like <code>fix-1.2.3.feature</code> are now
associated with the ticket <code>fix-1.2.3</code>.
In previous versions they were incorrectly associated to ticket
<code>3</code>.
(<code>[aio-libs#562](twisted/towncrier#562)
&lt;https://github.com/twisted/towncrier/issues/562&gt;</code>_)</li>
<li>Orphan newsfragments containing numeric values are no longer
accidentally associated to tickets. In previous versions the orphan
marker was ignored and the newsfragment was associated to a ticket
having the last numerical value from the filename.
(<code>[aio-libs#562](twisted/towncrier#562)
&lt;https://github.com/twisted/towncrier/issues/562&gt;</code>_)</li>
</ul>
<h2>Misc</h2>
<ul>
<li><code>[aio-libs#558](twisted/towncrier#558)
&lt;https://github.com/twisted/towncrier/issues/558&gt;</code><em>,
<code>[aio-libs#559](twisted/towncrier#559)
&lt;https://github.com/twisted/towncrier/issues/559&gt;</code></em></li>
</ul>
</blockquote>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a
href="https://github.com/twisted/towncrier/commit/503b499a1ddb13d6c526e5d5511767e53d809e68"><code>503b499</code></a>
Update final version.</li>
<li><a
href="https://github.com/twisted/towncrier/commit/bdb1d080e40babc7da54d3f6def37862d0cdcb94"><code>bdb1d08</code></a>
[pre-commit.ci] auto fixes from pre-commit.com hooks</li>
<li><a
href="https://github.com/twisted/towncrier/commit/47cf0f3b1d5f4be082fbddbad9b945274eb4d6bb"><code>47cf0f3</code></a>
Show release draft as a step output.</li>
<li><a
href="https://github.com/twisted/towncrier/commit/8b9b4b24136c628c874400ad1e9dd3a4a0a2c9bc"><code>8b9b4b2</code></a>
venv/bin/towncrier build --yes</li>
<li><a
href="https://github.com/twisted/towncrier/commit/3b27ce2e7ec2f219a8a655001bc21e5d4ba57412"><code>3b27ce2</code></a>
Fix orphan fragments with numbers (<a
href="https://redirect.github.com/twisted/towncrier/issues/564">#564</a>)</li>
<li><a
href="https://github.com/twisted/towncrier/commit/a275be976216b7e11b896b63df7fd9fcbccc0023"><code>a275be9</code></a>
[pre-commit.ci] pre-commit autoupdate (<a
href="https://redirect.github.com/twisted/towncrier/issues/563">#563</a>)</li>
<li><a
href="https://github.com/twisted/towncrier/commit/a794718b345cb332cd16ce9a807f4c02c68088d3"><code>a794718</code></a>
Update the documentation for the release process. (<a
href="https://redirect.github.com/twisted/towncrier/issues/558">#558</a>)</li>
<li><a
href="https://github.com/twisted/towncrier/commit/fba26a53022d6b5199a171018e0532a9ee59b596"><code>fba26a5</code></a>
Update build-and-inspect-python-package to 1.5.2 (<a
href="https://redirect.github.com/twisted/towncrier/issues/559">#559</a>)</li>
<li><a
href="https://github.com/twisted/towncrier/commit/e4b892fb21afce321a132780b7d4cf34814da7cb"><code>e4b892f</code></a>
Make consistent the expectation around non-existent fragments directory
(<a
href="https://redirect.github.com/twisted/towncrier/issues/557">#557</a>)</li>
<li><a
href="https://github.com/twisted/towncrier/commit/f6809f031fce9fe3568166247f8ccf8f7a6c4aaf"><code>f6809f0</code></a>
Release 23.10.0 (<a
href="https://redirect.github.com/twisted/towncrier/issues/555">#555</a>)</li>
<li>See full diff in <a
href="https://github.com/twisted/towncrier/compare/23.10.0...23.11.0">compare
view</a></li>
</ul>
</details>
<br />


[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=towncrier&package-manager=pip&previous-version=23.10.0&new-version=23.11.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after
your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge
and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating
it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)


</details>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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 a pull request may close this issue.

2 participants