Skip to content

Conversation

@fancyweb
Copy link
Contributor

Q A
Branch? 3.4
Bug fix? no
New feature? no
Deprecations? no
Tickets no
License MIT
Doc PR -

This property is unused. It was removed in #22943. It was likely reintroduced in a merge commit. It is still on master.

@chalasr chalasr added this to the 3.4 milestone Dec 13, 2019
@nicolas-grekas
Copy link
Member

Thank you @fancyweb.

nicolas-grekas added a commit that referenced this pull request Dec 13, 2019
…ncyweb)

This PR was merged into the 3.4 branch.

Discussion
----------

[SecurityBundle][FirewallMap] Remove unused property

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | no
| License       | MIT
| Doc PR        | -

This property is unused. It was removed in #22943. It was likely reintroduced in a merge commit. It is still on master.

Commits
-------

0904e57 [SecurityBundle][FirewallMap] Remove unused property
@nicolas-grekas nicolas-grekas merged commit 0904e57 into symfony:3.4 Dec 13, 2019
@fancyweb fancyweb deleted the security-bundle-remove-unused-property branch December 13, 2019 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants