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

Replace Namespace with Atom #3512

Closed
wants to merge 1 commit into from
Closed

Conversation

@kmcallister
Copy link
Contributor

kmcallister commented Sep 28, 2014

See discussion on kmcallister/html5ever#26.

This is part of the html5ever integration, but I'd like to get it in now to avoid excessive rebasing.

See discussion on html5ever #26
@hoppipolla-critic-bot
Copy link

hoppipolla-critic-bot commented Sep 28, 2014

Critic review: https://critic.hoppipolla.co.uk/r/2718

This is an external review system which you may optionally use for the code review of your pull request.

In order to help critic track your changes, please do not make in-place history rewrites (e.g. via git rebase -i or git commit --amend) when updating this pull request.

@highfive
Copy link

highfive commented Sep 28, 2014

warning Warning warning

  • These commits modify unsafe code. Please review it carefully!
@kmcallister
Copy link
Contributor Author

kmcallister commented Sep 30, 2014

I'll open a new PR with the type wrapper.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.