Skip to content
This repository has been archived by the owner on Apr 14, 2021. It is now read-only.

Rename Bundler.frozen? to avoid Object method conflict #6257

Merged
merged 1 commit into from Jan 16, 2018

Conversation

segiddins
Copy link
Member

What was the end-user problem that led to this PR?

The problem was Bundler overrode the Object#frozen? method to mean something else, breaking anything that checks that method to determine if an object is frozen.

Closes #6252.

What was your diagnosis of the problem?

My diagnosis was the method name had to be changed.

@segiddins segiddins added this to the 1.16.2 milestone Jan 15, 2018
@segiddins
Copy link
Member Author

@bundlerbot r+

@bundlerbot
Copy link
Collaborator

📌 Commit 59d21d5 has been approved by segiddins

@bundlerbot
Copy link
Collaborator

⌛ Testing commit 59d21d5 with merge 2c05914...

bundlerbot added a commit that referenced this pull request Jan 16, 2018
Rename Bundler.frozen? to avoid Object method conflict

### What was the end-user problem that led to this PR?

The problem was `Bundler` overrode the `Object#frozen?` method to mean something else, breaking anything that checks that method to determine if an object is frozen.

Closes #6252.

### What was your diagnosis of the problem?

My diagnosis was the method name had to be changed.
@bundlerbot
Copy link
Collaborator

☀️ Test successful - status-travis
Approved by: segiddins
Pushing 2c05914 to master...

@bundlerbot bundlerbot merged commit 59d21d5 into master Jan 16, 2018
@segiddins segiddins deleted the seg-rename-bundler-frozen branch January 16, 2018 02:34
colby-swandale pushed a commit that referenced this pull request Apr 11, 2018
Rename Bundler.frozen? to avoid Object method conflict

### What was the end-user problem that led to this PR?

The problem was `Bundler` overrode the `Object#frozen?` method to mean something else, breaking anything that checks that method to determine if an object is frozen.

Closes #6252.

### What was your diagnosis of the problem?

My diagnosis was the method name had to be changed.

(cherry picked from commit 2c05914)
colby-swandale pushed a commit that referenced this pull request Apr 20, 2018
Rename Bundler.frozen? to avoid Object method conflict

### What was the end-user problem that led to this PR?

The problem was `Bundler` overrode the `Object#frozen?` method to mean something else, breaking anything that checks that method to determine if an object is frozen.

Closes #6252.

### What was your diagnosis of the problem?

My diagnosis was the method name had to be changed.

(cherry picked from commit 2c05914)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants