Page MenuHomeDevCentral

Add plugin for terminator to resolve hashes
Needs ReviewPublic

Authored by Duranzed on Mon, Feb 23, 16:23.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Feb 26, 01:32
Unknown Object (File)
Tue, Feb 24, 12:12
Unknown Object (File)
Tue, Feb 24, 12:12
Unknown Object (File)
Tue, Feb 24, 10:30
Unknown Object (File)
Tue, Feb 24, 10:30
Unknown Object (File)
Tue, Feb 24, 10:30
Unknown Object (File)
Tue, Feb 24, 09:00
Unknown Object (File)
Tue, Feb 24, 08:49
Subscribers
None
Tokens
"Like" token, awarded by dereckson.

Details

Summary

This terminator plugin uses resolve-hash module
to solve then redirects to specified commit.

Ref T1695.

Diff Detail

Repository
rRESOLVEHASH Resolve hash
Lint
Lint Passed
Unit
No Test Coverage
Branch
terminator-plugin
Build Status
Buildable 6378
Build 6662: arc lint + arc unit

Event Timeline

Duranzed created this revision.

Not for the operations repository, it's for rRESOLVEHASH.

Repository move

@Duranzed Can you try to recreate the commit on a branch on the top of resolve-hash main and try this command?

arc diff --update D3976 HEAD^ -m 'Moving to resolve-hash repository'

dereckson requested changes to this revision.Mon, Feb 23, 17:57
This revision now requires changes to proceed.Mon, Feb 23, 17:57

Moving to resolve-hash repository

Moving to resolve-hash repository

Updated hash resolver plugin for terminator :

  • Intégration native de resolve-hash
  • Optimisation des performances
  • Gestion des erreurs

Plugin works, but we need more metadata:

  • header on the Python file
  • explanation in the README how to install it
support/terminator/resolve_hash.py
2

Header is missing.

I'd suggest something like

#   -------------------------------------------------------------
#   Resolve hash
#   - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
#   Project:        Nasqueron
#   Description:    Terminator plugin to call Resolve hash as
#                   URL handler for longer hashes
#   License:        BSD-2-Clause
#   -------------------------------------------------------------
dereckson requested changes to this revision.Wed, Feb 25, 23:42
This revision now requires changes to proceed.Wed, Feb 25, 23:42

Added README.md file for installation steps
Added Nasqueron header in the plugin file

Duranzed marked an inline comment as done.

Removed unused imports