Skip to content

Conversation

@solenko
Copy link

@solenko solenko commented Nov 23, 2016

All PRs:

  • Has tests
  • Documentation updated

When include ActionController::Base.include(::SecureHeaders) called from reiltie.rb,
SecuerHeaders.add_features not defined yet.
As a result class/instance methods won't added to ActionController::Base.

But introduced in commit 4aee15e

Wahen include ActionController::Base.include(::SecureHeaders) called from reiltie.rb,
SecuerHeaders.add_features not defined yet.

As a result class/instance methods won't added to ActionController::Base.

But introduced in commit 4aee15e
@solenko
Copy link
Author

solenko commented Nov 24, 2016

Build failure not related to my changes -- seems build for 2.x branch broken.
Do You plan to maintain this branch?

@oreoshake
Copy link
Contributor

oreoshake commented Nov 24, 2016 via email

@solenko solenko force-pushed the 2.x branch 4 times, most recently from 9fb504e to 07d0ec9 Compare November 24, 2016 19:31
@solenko
Copy link
Author

solenko commented Nov 24, 2016

Sorry, I forgot about US holydays. Build fixed.

Happy Thanksgiving!

@oreoshake oreoshake merged commit 56dc46d into github:2.x Nov 28, 2016
@oreoshake
Copy link
Contributor

Released in 2.5.3

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