Skip to content

Commit

Permalink
Remove incremental
Browse files Browse the repository at this point in the history
  • Loading branch information
SmileyChris committed Mar 27, 2023
1 parent 2891e01 commit 8667326
Show file tree
Hide file tree
Showing 17 changed files with 134 additions and 213 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -214,7 +214,7 @@ jobs:

- name: Install dependencies
run: |
python -m pip install --upgrade pip setuptools wheel
python -m pip install --upgrade pip wheel
python -m pip install pep517
- name: Display structure of files to be pushed
Expand Down
13 changes: 6 additions & 7 deletions RELEASE.rst
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,9 @@ The same branch is used for the release candidated and the final release.
In the end, the release branch is merged into the main branch.

Update the version to the release candidate with the first being ``rc1`` (as opposed to 0).
In ``src/towncrier/_version.py`` the version is set using ``incremental`` such as::
In ``pyproject.toml`` the version is set like::

__version__ = Version('towncrier', 19, 9, 0, release_candidate=1)
version = "19.9.0.rc1"

Run ``venv/bin/towncrier build --yes`` to generate the news release NEWS file.
Commit and push to the primary repository, not a fork.
Expand Down Expand Up @@ -59,9 +59,9 @@ Final release
Once the PR is approved, you can trigger the final release.

Update the version to the final version.
In ``src/towncrier/_version.py`` the version is set using ``incremental`` such as::
In ``pyproject.toml`` the version is set like::

__version__ = Version('towncrier', 19, 9, 0)
version = "19.9.0"

Manually update the `NEWS.rst` file to include the final release version and date.
Usually it will look like this::
Expand All @@ -88,10 +88,9 @@ Similar to the release candidate, with the difference:
No need for another review request.

Update the version to the development version.
In ``src/towncrier/_version.py`` the version is set using ``incremental`` such as::

__version__ = Version('towncrier', 19, 9, 1, dev=0)
In ``pyproject.toml`` the version is set like::

version = "19.9.1.dev0"

Commit and push the changes.

Expand Down
32 changes: 0 additions & 32 deletions admin/canonicalize_version.py

This file was deleted.

13 changes: 6 additions & 7 deletions docs/conf.py
Original file line number Diff line number Diff line change
Expand Up @@ -29,11 +29,12 @@
# Add any Sphinx extension module names here, as strings. They can be
# extensions coming with Sphinx (named 'sphinx.ext.*') or your custom
# ones.
from datetime import date
import importlib.metadata

from towncrier import __version__ as towncrier_version
from datetime import date


towncrier_version = importlib.metadata.version("towncrier")
extensions = []

# Add any paths that contain templates here, relative to this directory.
Expand All @@ -53,19 +54,17 @@
project = "Towncrier"
copyright = "{}, Towncrier contributors. Ver {}".format(
_today.year,
towncrier_version.public(),
importlib.metadata.version("towncrier"),
)
author = "Amber Brown"

# The version info for the project you're documenting, acts as replacement for
# |version| and |release|, also used in various other places throughout the
# built documents.
# The short X.Y version.
version = "{}.{}.{}".format(
towncrier_version.major, towncrier_version.minor, towncrier_version.micro
)
version = ".".join(towncrier_version.split(".")[:3])
# The full version, including alpha/beta/rc tags.
release = towncrier_version.public()
release = towncrier_version

# List of patterns, relative to source directory, that match files and
# directories to ignore when looking for source files.
Expand Down
2 changes: 1 addition & 1 deletion noxfile.py
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ def check_newsfragment(session: nox.Session) -> None:

@nox.session
def typecheck(session: nox.Session) -> None:
session.install(".", "mypy", "types-setuptools")
session.install(".", "mypy")
session.run("mypy", "src")


Expand Down
16 changes: 1 addition & 15 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -2,14 +2,13 @@
requires = [
"hatchling ~= 1.12.2",
"wheel ~= 0.38.4",
"incremental == 22.10.0",
]
build-backend = "hatchling.build"


[project]
dynamic = ["version"]
name = "towncrier"
version = "22.12.1.dev0"
description = "Building newsfiles for your project."
readme = "README.rst"
license = "MIT"
Expand All @@ -32,9 +31,7 @@ requires-python = ">=3.7"
dependencies = [
"click",
"click-default-group",
"incremental",
"jinja2",
"setuptools",
"tomli; python_version<'3.11'",
]

Expand All @@ -60,11 +57,6 @@ Coverage = "https://codecov.io/gh/twisted/towncrier"
Distribution = "https://pypi.org/project/towncrier"


[tool.hatch.version]
source = "code"
path = "src/towncrier/_version.py"
expression = "_hatchling_version"

[tool.hatch.build]
exclude = [
"admin",
Expand Down Expand Up @@ -148,12 +140,6 @@ module = 'click_default_group'
# 2022-09-04: This library has no type annotations.
ignore_missing_imports = true

[[tool.mypy.overrides]]
module = 'incremental'
# No released version with type hints.
ignore_missing_imports = true


[tool.coverage.run]
parallel = true
branch = true
Expand Down
26 changes: 0 additions & 26 deletions src/towncrier/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,29 +4,3 @@
"""
towncrier, a builder for your news files.
"""

from __future__ import annotations

from incremental import Version


__all__ = ["__version__"]


def __getattr__(name: str) -> Version:
if name != "__version__":
raise AttributeError(f"module {__name__} has no attribute {name}")

import warnings

from ._version import __version__

warnings.warn(
"Accessing towncrier.__version__ is deprecated and will be "
"removed in a future release. Use importlib.metadata directly "
"to query for towncrier's packaging metadata.",
DeprecationWarning,
stacklevel=2,
)

return __version__
54 changes: 26 additions & 28 deletions src/towncrier/_project.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,12 @@

from __future__ import annotations

import importlib.metadata
import sys

from importlib import import_module
from types import ModuleType

from incremental import Version as IncrementalVersion
from typing import Any


def _get_package(package_dir: str, package: str) -> ModuleType:
Expand All @@ -38,47 +38,45 @@ def _get_package(package_dir: str, package: str) -> ModuleType:


def get_version(package_dir: str, package: str) -> str:
module = _get_package(package_dir, package)
version: Any

module = _get_package(package_dir, package)
version = getattr(module, "__version__", None)

if not version:
raise Exception("No __version__, I don't know how else to look")
try:
version = importlib.metadata.version(package)
except importlib.metadata.PackageNotFoundError:
raise Exception(f"Package not installed and no {package}.__version__ found")

if isinstance(version, str):
return version.strip()

if isinstance(version, IncrementalVersion):
# FIXME:https://github.com/twisted/incremental/issues/81
# Incremental uses `.rcN`.
# importlib uses `rcN` (without a dot separation).
# Here we make incremental work like importlib.
return version.base().strip().replace(".rc", "rc")

if isinstance(version, tuple):
return ".".join(map(str, version)).strip()

# Try duck-typing as an Incremental version.
if hasattr(version, "base"):
try:
version = str(version.base()).strip()
# Incremental uses `X.Y.rcN`.
# Standardize on importlib (and PEP440) use of `X.YrcN`:
return version.replace(".rc", "rc") # type: ignore
except TypeError:
pass

raise Exception(
"I only know how to look at a __version__ that is a str, "
"an Increment Version, or a tuple. If you can't provide "
"that, use the --version argument and specify one."
"Version must be a string, tuple, or an Incremental Version."
" If you can't provide that, use the --version argument and specify one."
)


def get_project_name(package_dir: str, package: str) -> str:
module = _get_package(package_dir, package)

version = getattr(module, "__version__", None)
# Incremental has support for package names, try duck-typing it.
try:
return str(version.package) # type: ignore
except AttributeError:
pass

if not version:
# welp idk
return package.title()

if isinstance(version, str):
return package.title()

if isinstance(version, IncrementalVersion):
# Incremental has support for package names
return version.package

raise TypeError(f"Unsupported type for __version__: {type(version)}")
return package.title()
37 changes: 20 additions & 17 deletions src/towncrier/_settings/load.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,15 +3,15 @@

from __future__ import annotations

import importlib.resources
import os
import sys

from collections import OrderedDict
from dataclasses import dataclass
from pathlib import Path
from typing import TYPE_CHECKING, Any, Mapping

import pkg_resources

from .._settings import fragment_types as ft


Expand Down Expand Up @@ -145,23 +145,26 @@ def parse_toml(base_path: str, config: Mapping[str, Any]) -> Config:
)

template = config.get("template", _template_fname)
if template.startswith("towncrier:"):
resource_name = "templates/" + template.split("towncrier:", 1)[1] + ".rst"
if not pkg_resources.resource_exists("towncrier", resource_name):
raise ConfigError(
"Towncrier does not have a template named '%s'."
% (template.split("towncrier:", 1)[1],)
)

template = pkg_resources.resource_filename("towncrier", resource_name)
if ":" in template:
package, resource = template.split(":", 1)
if Path(resource).suffix not in (".rst", ".md"):
resource = resource + ".rst"
if not importlib.resources.is_resource(package, resource):
if not importlib.resources.is_resource(package + ".templates", resource):
raise ConfigError(
f"'{package}' does not have a template named '{resource}'.",
failing_option="template",
)
else:
package = package + ".templates"
template = ":".join((package, resource))
else:
template = os.path.join(base_path, template)

if not os.path.exists(template):
raise ConfigError(
f"The template file '{template}' does not exist.",
failing_option="template",
)
if not os.path.isfile(template):
raise ConfigError(
f"The template file '{template}' does not exist.",
failing_option="template",
)

return Config(
package=config.get("package", ""),
Expand Down
11 changes: 9 additions & 2 deletions src/towncrier/_shell.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,18 +9,25 @@

from __future__ import annotations

from importlib.metadata import PackageNotFoundError, version

import click

from click_default_group import DefaultGroup

from ._version import __version__
from .build import _main as _build_cmd
from .check import _main as _check_cmd
from .create import _main as _create_cmd


try:
_version = version("towncrier")
except PackageNotFoundError:
_version = "unknown"


@click.group(cls=DefaultGroup, default="build", default_if_no_args=True)
@click.version_option(__version__.public())
@click.version_option(_version)
def cli() -> None:
"""
Towncrier is a utility to produce useful, summarised news files for your project.
Expand Down
16 changes: 0 additions & 16 deletions src/towncrier/_version.py

This file was deleted.

Loading

0 comments on commit 8667326

Please sign in to comment.