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

CSV support #5

Merged
merged 4 commits into from
Feb 25, 2015
Merged

CSV support #5

merged 4 commits into from
Feb 25, 2015

Conversation

orta
Copy link
Member

@orta orta commented Feb 20, 2015

Adds an option for outputting as CSV.
Considering this does everything it is set out to do, I've moved it to 1.0.0 too. No need for super low releases when it fits its scope correctly.

@orta
Copy link
Member Author

orta commented Feb 20, 2015

Assigning to you @kapin

when :md
UI.puts "* [#{pod[:name]} - #{pod[:version]}](#{pod[:homepage]}) [#{pod[:license][:type]}] - #{pod[:summary]}"
when :csv
UI.puts "#{pod[:name]}, #{pod[:version]}, #{pod[:homepage]}, \"#{pod[:summary]}\", \"#{pod[:license][:type]}\""
Copy link
Member

Choose a reason for hiding this comment

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

Why the spaces?

Copy link
Member Author

Choose a reason for hiding this comment

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

legit point, that was me trying to read it better, but it's not legit CSV

@joshkalpin
Copy link
Member

I'll take a proper look later tonight

@@ -15,14 +15,18 @@ class PodfileInfo < Command
def self.options
[
["--all", "Show information about all Pods with dependencies that are used in a project"],
["--md", "Output information in Markdown format"]
["--md", "Output information in Markdown format"],
["--csv", "Output information in Markdown format"]
Copy link
Member

Choose a reason for hiding this comment

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

This should read in CSV format

@joshkalpin
Copy link
Member

Just the one comment otherwise LGTM and I'll update the gem when I get home

@joshkalpin
Copy link
Member

Ping @orta

@orta
Copy link
Member Author

orta commented Feb 25, 2015

oops, sorry sure :D

@orta
Copy link
Member Author

orta commented Feb 25, 2015

Done

joshkalpin pushed a commit that referenced this pull request Feb 25, 2015
@joshkalpin joshkalpin merged commit f460f6a into master Feb 25, 2015
@joshkalpin
Copy link
Member

I'll update the gem tonight

@kylef kylef deleted the csv branch February 25, 2015 22:03
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.

3 participants