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

Fix homepage uri in gemspec. #36

Merged
merged 1 commit into from
Mar 13, 2015
Merged

Fix homepage uri in gemspec. #36

merged 1 commit into from
Mar 13, 2015

Conversation

JuanitoFatas
Copy link
Contributor

😉

@JuanitoFatas
Copy link
Contributor Author

Also could you please also add some metadata information of this gem in this page https://rubygems.org/gems/rails-html-sanitizer/edit 🙏:

Here are some I know:

URL VALUE
Source Code URL https://github.com/rails/rails-html-sanitizer
Documentation URL http://www.rubydoc.info/gems/rails-html-sanitizer
Bug Tracker URL https://github.com/rails/rails-html-sanitizer/issues
Wiki URL https://github.com/rails/rails-html-sanitizer/wiki

Thank you!

kaspth added a commit that referenced this pull request Mar 13, 2015
@kaspth kaspth merged commit 7cbe2d3 into rails:master Mar 13, 2015
@kaspth
Copy link
Contributor

kaspth commented Mar 13, 2015

Thanks. I don't have access to the gem on rubygems so someone else has to do that.

@JuanitoFatas JuanitoFatas deleted the fix/gemspec branch March 13, 2015 06:24
@JuanitoFatas
Copy link
Contributor Author

Thanks. I don't have access to the gem on rubygems so someone else has to do that.

OK!

@JuanitoFatas
Copy link
Contributor Author

@kaspth Should keep this issue open till the metadata are done?

@kaspth
Copy link
Contributor

kaspth commented Mar 13, 2015

It's a pull request, so we can't reopen. Let's just wait until people on the other side of the world wake up 😄

@JuanitoFatas
Copy link
Contributor Author

@kaspth Ah right, it is a pull request....... hahaha! 😅

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.

2 participants