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

Fixing IE tests. Shows that tests still fail for 3780bf59 #49

Merged
merged 1 commit into from Jun 4, 2014
Merged

Fixing IE tests. Shows that tests still fail for 3780bf59 #49

merged 1 commit into from Jun 4, 2014

Conversation

ibash
Copy link
Contributor

@ibash ibash commented Jun 2, 2014

Tests added here: 3780bf5

And some previous tests are broken. See this comment to see why: #44 (comment)

@ibash
Copy link
Contributor Author

ibash commented Jun 4, 2014

cc @evilpacket / @EvanHahn
This is still broken -- can you either take in the fixes from the closed pull request or at least fix the tests (so it's known that it's broken).

Let me know if you want another pull request actually fixing the bug as well.

EvanHahn added a commit that referenced this pull request Jun 4, 2014
Fixing IE tests. Shows that tests still fail for 3780bf5
@EvanHahn EvanHahn merged commit da9de69 into helmetjs:master Jun 4, 2014
EvanHahn added a commit that referenced this pull request Jun 4, 2014
@EvanHahn
Copy link
Member

EvanHahn commented Aug 5, 2016

I'm making a new website for Helmet.js and I want to credit everyone who's contributed. Do you have a name and/or website you'd like me to use to credit you?

I'll credit you for this PR and #44, too.

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

Successfully merging this pull request may close these issues.

None yet

2 participants