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

Backporting #1245 (Unpermitted parameters) to 7.2.x #1271

Merged
merged 7 commits into from Aug 23, 2016

Conversation

ProGM
Copy link
Member

@ProGM ProGM commented Aug 22, 2016

Backporting #1245 to 7.x

I'm opening a PR since there where a few conflicts.

Pings:
@cheerfulstoic
@subvertallchris

@cheerfulstoic
Copy link
Contributor

Seems good to me, but I think that this would be when you add functionality in a backwards-compatible manner which would be a minor update according to semver.org

Shall we release as 7.2? I've been trying to release patch versions whenever there's something to be released which leads us up to .15 sometimes, but I feel like we hold back on minor updates

@ProGM
Copy link
Member Author

ProGM commented Aug 23, 2016

Since this fixes this bug:
#1244

I considered it as a "bugfix", but instead it's a feature.

So, for me it's ok to release a 7.2.x version ;)

@codecov-io
Copy link

codecov-io commented Aug 23, 2016

Current coverage is 97.24% (diff: 100%)

Merging #1271 into 7.1.x will increase coverage by 0.01%

@@              7.1.x      #1271   diff @@
==========================================
  Files           168        170     +2   
  Lines         10426      10482    +56   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits          10137      10193    +56   
  Misses          289        289          
  Partials          0          0          

Powered by Codecov. Last update abc2cbc...f7ae0f1

@ProGM ProGM changed the base branch from 7.1.x to 7.2.x August 23, 2016 14:09
@ProGM ProGM changed the title Backporting #1245 (Unpermitted parameters) to 7.1.x Backporting #1245 (Unpermitted parameters) to 7.2.x Aug 23, 2016
@ProGM ProGM merged commit 2b39b2d into 7.2.x Aug 23, 2016
@ProGM ProGM deleted the fix-active-controller-params-7.1.x branch August 23, 2016 14:10
@ProGM ProGM removed the in progress label Aug 23, 2016
ProGM added a commit that referenced this pull request Aug 23, 2016
ProGM added a commit that referenced this pull request Aug 23, 2016
* 7.2.x:
  Bump version 7.2.0
  CHANGELOG entry for #1271
  Another try...
  Trying to fix gemfile.
  Refactor `using_action_controller` to print output.
  Fix typo.
  ActiveRel improvements to make permitted_attributes work on it.
  Make changes work with rails 4.x
  Support for Unpermitted Attributes API.
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.

None yet

3 participants