Skip to content

Document dependency on Mail::SPF#1225

Merged
marc-vanderwal merged 1 commit into
zonemaster:developfrom
marc-vanderwal:feature/depend-on-mail-spf
Dec 4, 2023
Merged

Document dependency on Mail::SPF#1225
marc-vanderwal merged 1 commit into
zonemaster:developfrom
marc-vanderwal:feature/depend-on-mail-spf

Conversation

@marc-vanderwal

Copy link
Copy Markdown
Contributor

Purpose

This PR updates installation instructions with a new dependency made necessary by zonemaster/zonemaster-engine#1287.

Context

Preparation of pull request zonemaster/zonemaster-engine#1287.

Changes

Document the extra dependency on Mail::SPF at run-time.

How to test this PR

Documentation update.

Comment thread docs/public/installation/zonemaster-engine.md
@marc-vanderwal marc-vanderwal force-pushed the feature/depend-on-mail-spf branch from 0fb13ea to 0857adf Compare November 28, 2023 09:44
matsduf
matsduf previously approved these changes Nov 28, 2023
tgreenx
tgreenx previously approved these changes Nov 28, 2023
mattias-p
mattias-p previously approved these changes Nov 30, 2023
Pull request zonemaster/zonemaster-engine#1287 adds a run-time
dependency on Mail::SPF. This comment documents the change in the
public-facing installation instructions.

Mail::SPF is not required for building Zonemaster::Engine, so the
internal documentation pertaining to the build environment does not need
to be updated.
@marc-vanderwal marc-vanderwal dismissed stale reviews from mattias-p, tgreenx, and matsduf via 7c5a755 December 4, 2023 11:07
@marc-vanderwal marc-vanderwal force-pushed the feature/depend-on-mail-spf branch from 0857adf to 7c5a755 Compare December 4, 2023 11:07
@matsduf

matsduf commented Dec 4, 2023

Copy link
Copy Markdown
Contributor

@marc-vanderwal, please merge.

@marc-vanderwal marc-vanderwal merged commit 0bdf383 into zonemaster:develop Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Documentation Area: Documentation only.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants