Skip to content
This repository has been archived by the owner on Nov 30, 2024. It is now read-only.

Use configuration rather than streams directly in deprecation formatter and fall back to std pipes #1130

Merged
merged 4 commits into from
Nov 2, 2013

Conversation

JonRowe
Copy link
Member

@JonRowe JonRowe commented Oct 30, 2013

Potential solution to #1128 rebased off 2-99

@myronmarston
Copy link
Member

This looks good except for the failing build. Let me know if you want help with that.

@myronmarston
Copy link
Member

@JonRowe -- can you fix the failing build and get this ready to merge in the next day or two? If not, let me know and I'll take care of it. My in-progress fix for #1092 is blocked until this is merged, and our 3.0 alpha release is blocked until #1092 is done.

@JonRowe
Copy link
Member Author

JonRowe commented Nov 1, 2013

@myronmarston done

@JonRowe
Copy link
Member Author

JonRowe commented Nov 2, 2013

or maybe not...

@myronmarston
Copy link
Member

Thanks Jon...merging since it's green so I can work off this for my PR.

myronmarston added a commit that referenced this pull request Nov 2, 2013
Use configuration rather than streams directly in deprecation formatter and fall back to std pipes
@myronmarston myronmarston merged commit c63dc74 into 2-99-maintenance Nov 2, 2013
@myronmarston myronmarston deleted the deprecation_formatter_fix_299 branch November 2, 2013 00:31
@JonRowe
Copy link
Member Author

JonRowe commented Nov 2, 2013

👍

myronmarston added a commit that referenced this pull request Nov 3, 2013
…_299"

This reverts commit c63dc74, reversing
changes made to 64ba913.

We're reverting this because this didn't actually solve the problem
of issueing deprecation warnings from option_parser.rb.
yujinakayama pushed a commit to yujinakayama/rspec-monorepo that referenced this pull request Oct 6, 2021
…n_formatter_fix_299

Use configuration rather than streams directly in deprecation formatter and fall back to std pipes

---
This commit was imported from rspec/rspec-core@c63dc74.
yujinakayama pushed a commit to yujinakayama/rspec-monorepo that referenced this pull request Oct 6, 2021
…precation_formatter_fix_299"

This reverts commit c63dc74b8b4b5f5f7ba89aa5405ee862130818f2, reversing
changes made to 64ba91330df9e8d8cd47f7831e25f19089af4361.

We're reverting this because this didn't actually solve the problem
of issueing deprecation warnings from option_parser.rb.

---
This commit was imported from rspec/rspec-core@fdcd136.
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.

2 participants