The core Mercurial manifest template prints the rev number and the short hash.
Since treemanifest doesn't have rev numbers, this has to change. Let's just have
it print the whole hash, since manifest hashes are usually only ever used by
automation which probably wants the whole hash anyway.
Details
Details
- Reviewers
quark - Group Reviewers
Restricted Project - Commits
- rFBHGX2dd048101c46: treemanifest: support the {manifest} template
Diff Detail
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
treemanifest/__init__.py | ||
---|---|---|
163 | This breaks {manifest % "{node}"}. Maybe add a comment saying we don't care about that use-case? |
treemanifest/__init__.py | ||
---|---|---|
163 | I didn't realize that was even possible. Do we use that anywhere? |
treemanifest/__init__.py | ||
---|---|---|
163 | I think we didn't use that. The flexibility is for people wanting {rev} or {node}. I think it's fine to not support that with a comment noting the slight difference. |
This breaks {manifest % "{node}"}. Maybe add a comment saying we don't care about that use-case?