Differential D4660

[Nix] Rephrase motivation behind pinning to better reflect upstream
ClosedPublic
Actions

Authored by • jon on Jul 28 2022, 12:13 AM.

Details

Reviewers

ashoat

Commits

rCOMMdbe97ad840bf: [Nix] Rephrase motivation behind pinning to better reflect upstream

Summary

Address https://phab.comm.dev/D4646#133604

Previous wording reduced options to only the await option. When json imports
can also use asserts. Instead, better align with upstream language.

Test Plan

N/A

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

• jon created this revision.Jul 28 2022, 12:13 AM

Herald added subscribers: • abosh, atul, • adrian, tomek. · View Herald TranscriptJul 28 2022, 12:13 AM

Harbormaster completed remote builds in B10924: Diff 15030.Jul 28 2022, 12:23 AM

• jon requested review of this revision.Jul 28 2022, 12:23 AM

• jon mentioned this in D4646: [Nix] Update nixpkgs pin.Jul 28 2022, 12:48 AM

• jon added a child revision: D4646: [Nix] Update nixpkgs pin.Jul 28 2022, 1:11 AM

• jon edited the summary of this revision. (Show Details)

Thank you for fixing this!

This revision is now accepted and ready to land.Jul 28 2022, 2:37 AM

Closed by commit rCOMMdbe97ad840bf: [Nix] Rephrase motivation behind pinning to better reflect upstream. · Explain WhyJul 28 2022, 11:58 PM

This revision was automatically updated to reflect the committed changes.

• jon added a commit: rCOMMdbe97ad840bf: [Nix] Rephrase motivation behind pinning to better reflect upstream.

Revision Contents
Changeset List

Path

Size

nix/

overlay.nix

2 lines

Diff 15073

View Options

[Nix] Rephrase motivation behind pinning to better reflect upstreamClosedPublicActions