This is an archive of the discontinued Mercurial Phabricator instance.

configs: register more of our configs
ClosedPublic

Authored by durham on Oct 23 2017, 3:06 PM.
Tags
None
Subscribers
None

Details

Reviewers
quark
Group Reviewers
Restricted Project
Commits
rFBHGX2489894d7567: configs: register more of our configs
Summary

Upstream has added devel warnings that require us to register all the configs
and remove the defaults that are specified at read time. This doesn't fix all
the cases, but is the start.

Diff Detail

Repository
rFBHGX Facebook Mercurial Extensions
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

durham created this revision.Oct 23 2017, 3:06 PM
Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 23 2017, 3:06 PM
durham updated this revision to Diff 3055.Oct 23 2017, 3:09 PM
durham added inline comments.Oct 23 2017, 3:10 PM
tests/test-fbamend-to.t
8

This was needed because fbamend calls histedit.histedit which requires that the histedit extension be configured (because it needs to register it's configs).

quark accepted this revision.Oct 23 2017, 4:50 PM
This revision is now accepted and ready to land.Oct 23 2017, 4:50 PM
This revision was automatically updated to reflect the committed changes.