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

modified: README #1

Merged
merged 1 commit into from Apr 28, 2015

Conversation

Projects
None yet
2 participants
@internaught
Contributor

internaught commented Apr 28, 2015

modified:   README.mkdn
modified:   README.pod
modified:   lib/WebService/UrbanDictionary/Term.pm
modified:   lib/WebService/UrbanDictionary/Term/Definition.pm

The synopsis was broken in that the line: my $results = request('perl'); would fail with errors.

Changing the line to my $results = $ud->request('perl'); makes the example work.

The synopsis for WebService::UrbanDictionary::Term and WebService::UrbanDictionary::Term::Definition should also be changed to reflect the same.

I think there could be more examples for this module and it's missing documentation for "definitions".

modified: README
	modified:   README.mkdn
	modified:   README.pod
	modified:   lib/WebService/UrbanDictionary/Term.pm
	modified:   lib/WebService/UrbanDictionary/Term/Definition.pm

woodruffw added a commit that referenced this pull request Apr 28, 2015

@woodruffw woodruffw merged commit eef3ee2 into woodruffw:master Apr 28, 2015

@woodruffw

This comment has been minimized.

Show comment
Hide comment
@woodruffw

woodruffw Apr 28, 2015

Owner

Thanks. Merged.

Owner

woodruffw commented Apr 28, 2015

Thanks. Merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment