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

Shields gem #27

Closed
wants to merge 6 commits into from
Closed

Shields gem #27

wants to merge 6 commits into from

Conversation

olivierlacan
Copy link
Member

Opening a quick pull request to start working on an initial gem to generate badge PNGs from the SVG source with @ackerdev.

The branch is called API because we're way too fucking ambitious with this.

Tasks

  • get a basic CLI workflow to generate one shield

@olivierlacan
Copy link
Member Author

After talking to @ackerdev, it seems like we should stop fiddling around with the CLI interface work and get a basic proof of concept working.

And it seems like using an additional text layer instead of a filter to generate the text shadow will have better results.

One possible way to deal with multiple versions would be a ShieldTemplate class, used like this:

ShieldTemplate.new({ label: "Code Climate", variations: [1.0, ..., 4.0]})

Or something like that.

@@ -0,0 +1,16 @@
#!/usr/bin/env /Users/olivierlacan/.bundle/ruby-env
Copy link

Choose a reason for hiding this comment

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

hard coded to your path

Copy link
Member Author

Choose a reason for hiding this comment

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

Yep, normal. Should have .gitignored all that stuff.

@ackerdev ackerdev mentioned this pull request May 2, 2013
3 tasks
@olivierlacan
Copy link
Member Author

I'm starting to be ashamed of this thing.

@chadwhitacre
Copy link
Contributor

This would theoretically back the service (#15), yes?

@olivierlacan
Copy link
Member Author

Closing this since we've abandoned this initiative.

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.

None yet

3 participants