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

Updates from rspec-dev (2016-07-14) #2293

Merged
merged 5 commits into from
Jul 19, 2016

Conversation

JonRowe
Copy link
Member

@JonRowe JonRowe commented Jul 13, 2016

These are some updates, generated from rspec-dev's rake tasks.

@JonRowe JonRowe force-pushed the update-travis-build-scripts-2016-07-14-for-master branch from aed781d to b32a5cc Compare July 15, 2016 12:55
@JonRowe JonRowe force-pushed the update-travis-build-scripts-2016-07-14-for-master branch from baa5250 to b985c4f Compare July 17, 2016 23:39
@JonRowe
Copy link
Member Author

JonRowe commented Jul 19, 2016

The same broken backtrace filtering that had to be pended in the specs needed to be pended in the cukes, ok to merge @myronmarston ?

@@ -28,7 +28,7 @@ function run_specs_and_record_done {

# rspec-core needs to run with a special script that loads simplecov first,
# so that it can instrument rspec-core's code before rspec-core has been loaded.
if [ -f script/rspec_with_simplecov ]; then
if [ -f script/rspec_with_simplecov ] && is_mri; then
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does simplecov not work on JRuby anymore?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure we've ever tried to run it before, our other JRuby modes report themselves as < 2 so miss this anyway, but rspec-core is reporting 0% coverage on JRuby 9.1.50

@myronmarston
Copy link
Member

Hmm, there's kinda a lot that you had to pend here. I don't want these things to be marked as pending and just be left that way. Do you have a plan for tackling them?

@JonRowe
Copy link
Member Author

JonRowe commented Jul 19, 2016

Yeah I wasn't overly happy doing it but I feel as this is a build steps PR and I didn't have the time to fix them straight away, would you be happy if I opened an issue to keep track of them?

@myronmarston
Copy link
Member

Yeah I wasn't overly happy doing it but I feel as this is a build steps PR and I didn't have the time to fix them straight away, would you be happy if I opened an issue to keep track of them?

Sure. It'd be great if the issues provided some detail about what the failure is.

@JonRowe
Copy link
Member Author

JonRowe commented Jul 19, 2016

Ok to merge otherwise?

@myronmarston
Copy link
Member

Ok to merge otherwise?

Yes

@JonRowe JonRowe merged commit e752392 into master Jul 19, 2016
@JonRowe JonRowe deleted the update-travis-build-scripts-2016-07-14-for-master branch July 19, 2016 23:57
MatheusRich pushed a commit to MatheusRich/rspec-core that referenced this pull request Oct 30, 2020
…2016-07-14-for-master

Updates from rspec-dev (2016-07-14)
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