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

hyrax 2.0 Beta4 #149

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

hyrax 2.0 Beta4 #149

wants to merge 1 commit into from

Conversation

VivianChu
Copy link
Member

Fixes #145 ; refs #145

Evaluate Hyrax 2.0 beta4 (clean) vs Horton as-is.

@ucsdlib/developers - please review

@@ -121,5 +138,51 @@

config.example_status_persistence_file_path = 'spec/examples.txt'

# config.profile_examples = 10
=begin

Choose a reason for hiding this comment

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

Style/BlockComments: Do not use block comments.

config.shared_context_metadata_behavior = :apply_to_host_groups

# The settings below are suggested to provide a good initial experience
# with RSpec, but feel free to customize to your heart's content.

Choose a reason for hiding this comment

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

Layout/CommentIndentation: Incorrect indentation detected (column 0 instead of 2).

# a real object. This is generally recommended, and will default to
# `true` in RSpec 4.
mocks.verify_partial_doubles = false
#mocks.syntax = :expect

Choose a reason for hiding this comment

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

Layout/LeadingCommentSpace: Missing space after #.

unless example.metadata[:type] == :view || example.metadata[:no_clean]
ActiveFedora::Cleaner.clean!
end
end

Choose a reason for hiding this comment

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

Layout/TrailingWhitespace: Trailing whitespace detected.

# arbitrary gems may also be filtered via:
# config.filter_gems_from_backtrace("gem name")

config.before :each do |example|

Choose a reason for hiding this comment

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

RSpec/HookArgument: Omit the default :each argument for RSpec hooks.

config.filter_rails_from_backtrace!
# arbitrary gems may also be filtered via:
# config.filter_gems_from_backtrace("gem name")

Choose a reason for hiding this comment

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

Layout/TrailingWhitespace: Trailing whitespace detected.

use_extension(Hydra::ContentNegotiation)

use_extension( Hydra::ContentNegotiation )

Choose a reason for hiding this comment

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

Layout/TrailingWhitespace: Trailing whitespace detected.

@@ -27,8 +27,8 @@ class SolrDocument

# Do content negotiation for AF models.

use_extension(Hydra::ContentNegotiation)

use_extension( Hydra::ContentNegotiation )

Choose a reason for hiding this comment

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

Layout/SpaceInsideParens: Space inside parentheses detected.

@VivianChu VivianChu force-pushed the feature/evaluate_hyrax4 branch from 2fd55c5 to 9c64435 Compare November 28, 2017 23:55
Update ruby version

Fix rubocop defenses

Fix rubocop defenses
@VivianChu VivianChu force-pushed the feature/evaluate_hyrax4 branch from 9c64435 to ae09f5c Compare November 29, 2017 00:40
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.

2 participants