Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add show command #1

Merged
merged 1 commit into from
Jan 31, 2018
Merged

Add show command #1

merged 1 commit into from
Jan 31, 2018

Conversation

marcosgz
Copy link
Owner

Show real name of one or more obfuscated files

Usage:

obfusc show <obfuscated file or directory>

Files:
source: Relative or absolute directory where obfuscated files are stored. (Default to current directory)
TEXT
puts usage
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not write to stdout. Use Rails's logger if you want to log.

def run
files.each do |from, to|
puts "#{from}:"
puts "---> #{to}"
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not write to stdout. Use Rails's logger if you want to log.


def run
files.each do |from, to|
puts "#{from}:"
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do not write to stdout. Use Rails's logger if you want to log.

module Obfusc
# Perform tasks related `show` command
class ShowCommand

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra empty line detected at class body beginning.

@marcosgz
Copy link
Owner Author

Ebert has finished reviewing this Pull Request and has found:

  • 4 possible new issues (including those that may have been commented here).

You can see more details about this review at https://ebertapp.io/github/marcosgz/obfusc/pulls/1.

@marcosgz marcosgz merged commit 4fcc291 into master Jan 31, 2018
@marcosgz marcosgz deleted the show-command branch January 31, 2018 01:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant