Skip to content
This repository has been archived by the owner on Jan 10, 2023. It is now read-only.

Modulesync 0.19.3 and more #40

Merged
merged 8 commits into from
Jan 20, 2017
Merged

Modulesync 0.19.3 and more #40

merged 8 commits into from
Jan 20, 2017

Conversation

vinzent
Copy link
Contributor

@vinzent vinzent commented Jan 20, 2017

Seems tests are failing since 1 year.

Unfortunatly I needed to skip some rspec tests. they work on my computer but not on travis. no idea what cold be wrong. someone with more ruby/augeas knowledge needs to have a look. I opend issue #41

Closes #39

@vinzent vinzent changed the title Modulesync 0.19.3 Modulesync 0.19.3 and more Jan 20, 2017

# Offense count: 2
# Configuration parameters: AssignmentOnly.
RSpec/InstanceVariable:
Copy link
Member

Choose a reason for hiding this comment

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

this whole block will be overwritten by the next modulesync

Copy link
Member

Choose a reason for hiding this comment

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

maybe disable the cops inside of the spec file?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

there is more overriden next modulesync. all in f711d50

if you know a better way just fix it yourself. :-)

@bastelfreak
Copy link
Member

thanks \o/

@bastelfreak bastelfreak merged commit 167d91f into voxpupuli:master Jan 20, 2017
@vinzent vinzent deleted the modulesync_0_19_3 branch February 3, 2017 21:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working enhancement New feature or request modulesync
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants