Skip to content
This repository has been archived by the owner on Oct 15, 2022. It is now read-only.

First Goodie hack #1235

Closed
wants to merge 6 commits into from
Closed

First Goodie hack #1235

wants to merge 6 commits into from

Conversation

abhijainn27
Copy link

Goodie Pull Request Template

Using this template will help us better understand your Instant Answer and assist you when necessary. Simply copy and paste the markdown below into the description of your GitHub pull request and complete as appropriate.

What does your Instant Answer do?
It prints my github username and tell that am awesome and completed the goodie hack.

What problem does your Instant Answer solve (Why is it better than organic links)?
It helps to find me among all other guthub users

What is the data source for your Instant Answer? (Provide a link if possible)
https://duck.co/duckduckhack/goodie_quickstart

Why did you choose this data source?
Because it is my first goodie hack

Are there any other alternative (better) data sources?

What are some example queries that trigger this Instant Answer?
duckduckhack abhijainn27

Which communities will this Instant Answer be especially useful for? (gamers, book lovers, etc)
github users

Is this Instant Answer connected to a DuckDuckHack Instant Answer idea?

Which existing Instant Answers will this one supercede/overlap with?
None

Are you having any problems? Do you need our help with anything?
No

Where did you hear about DuckDuckHack? (For first time contributors)
FSMK

What does the Instant Answer look like? (Provide a screenshot for new or updated Instant Answers)
ddg

Checklist

Please place an 'X' where appropriate.

- [] Added metadata and attribution information (https://duck.co/duckduckhack/metadata)
- [x] Wrote test file and added to t/ directory (https://duck.co/duckduckhack/goodie_tests)
- [] Verified that instant answer adheres to design guidelines (https://duck.co/duckduckhack/design_styleguide)
- [] Verified that instant answer adheres to code styleguide (https://duck.co/duckduckhack/code_styleguide)
- [] Tested cross-browser compatibility

    Please let us know which browsers/devices you've tested on:
    - Windows 8
        - [] Google Chrome
        - [] Firefox
        - [] Opera
        - [] IE 10

    - Windows 7
        - [] Google Chrome
        - [] Firefox
        - [] Opera
        - [] IE 8
        - [] IE 9
        - [] IE 10

    - Windows XP
        - [] IE 7
        - [] IE 8

    - Mac OSX
        - [] Google Chrome
        - [] Firefox
        - [] Opera
        - [] Safari

    - iOS (iPhone)
        - [] Safari Mobile
        - [] Google Chrome
        - [] Opera

    - iOS (iPad)
        - [] Safari Mobile
        - [] Google Chrome
        - [] Opera

    - Android
        - [] Firefox
        - [] Native Browser
        - [] Google Chrome
        - [] Opera

    - Linux (Ubuntu)
        - [x] Firefox

# Metadata. See https://duck.co/duckduckhack/metadata for help in filling out this section.
name "IsAwesome abhijainn27";
description "Succinct explanation of what this instant answer does";
primary_example_queries "first example query", "second example query";
Copy link
Member

Choose a reason for hiding this comment

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

@abhijainn27 Could you change this to be the actual query the goodie will trigger with? In this case that would be "duckduckhack abhijainn27". Thanks!

@MariagraziaAlastra
Copy link
Member

@abhijainn27 Thanks for this! I've added a few comments inline, just minor things. Nice work!

@abhijainn27
Copy link
Author

Hey @MariagraziaAlastra I have made the above changes.. Should i send a new pull request ?

@MrChrisW
Copy link
Collaborator

MrChrisW commented Jul 6, 2015

@abhijainn27 Provided you push to your master branch, changes you make will be added to this PR.

# Metadata. See https://duck.co/duckduckhack/metadata for help in filling out this section.
name "IsAwesome abhijainn27";
description "Succinct explanation of what this instant answer does";
primary_example_queries "duckduckhack abhijainn27";secondary_example_queries "optional -- demonstrate any additional triggers";
Copy link
Collaborator

Choose a reason for hiding this comment

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

You can remove secondary_example_queries "optional -- demonstrate any additional triggers";

@moollaza moollaza self-assigned this Jul 6, 2015
@abhijainn27
Copy link
Author

Hey @MrChrisW I removed the secondary query and comment lines. Hope this works fine now. :)


return unless $_; # Guard against "no answer"

return $_;
Copy link
Collaborator

Choose a reason for hiding this comment

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

These two return statements are not needed. They'll never be reached as there's a return on line 24

@abhijainn27
Copy link
Author

Hey @MrChrisW I made the above changes.

@@ -0,0 +1,28 @@
package DDG::Goodie::IsAwesome::abhijainn27;

Copy link
Collaborator

Choose a reason for hiding this comment

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

@javathunderman
Copy link
Collaborator

Hey @abhijainn27, I left a line note for you. Please fix this and it should be ready to merge.

cc @MrChrisW

@abhijainn27
Copy link
Author

Hey @javathunderman I added the abstract.

@javathunderman
Copy link
Collaborator

@MrChrisW This LGTM...I suggest you give a final review.

name "IsAwesome abhijainn27";
description "My First Goodie Hack !! ";
primary_example_queries "duckduckhack abhijainn27";

Copy link
Collaborator

Choose a reason for hiding this comment

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

@abhijainn27 One last thing, please add

category "special";
topics "special_interest", "geek";

Like this

@abhijainn27
Copy link
Author

Done :)

package DDG::Goodie::IsAwesome::abhijainn27;
# ABSTRACT: abhijainn27 first Goodie Hack !!
# CATEGORY "DuckDuckHack Goodie";
# TOPICS "Web Development", "geek";
Copy link
Collaborator

Choose a reason for hiding this comment

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

@abhijainn27 Sorry for the delay.

You'll need to add these on line 14 exactly how they're in the comment, these are set meta types

Line 14/15

category "special";
topics "special_interest", "geek";

@moollaza moollaza assigned MrChrisW and unassigned moollaza Jul 23, 2015
@GuiltyDolphin
Copy link
Member

This is really old... I'll close this for now.

@abhijainn27 If you want to finish this, just let us know.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants