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

Commit

Permalink
Add some contibuting docs
Browse files Browse the repository at this point in the history
  • Loading branch information
rodjek committed Jun 9, 2013
1 parent b1f6aa1 commit 365f142
Show file tree
Hide file tree
Showing 18 changed files with 53 additions and 30 deletions.
4 changes: 1 addition & 3 deletions .travis.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
script: ./script/cibuild
rvm: 1.8.7
branches:
only:
- master
notifications:
email:
- [email protected]
Expand Down
28 changes: 28 additions & 0 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,28 @@
Thanks for contributing to puppet-logrotate! A couple of notes to help you out:

* Please don't bump the module version number.
* Make sure your changes have tests.
* Keep your pull requests on topic. Pull requests with a bunch of unrelated
changes won't get merged. Feel free to open seperate pull requests for the
other changes though.
* Yes, I commit vendored gem files into my repositories. No, this is not
a mistake. Please don't "fix" this in your pull request :)

Before starting, you can prepare your development environment by running:

```
script/bootstap
```

This will install all the dependencies required to run the test suite.

Although TravisCI will automatically run the test suite against your branch
when you push, you can (and should) run them locally as you're working. You can
run the test suite by running:

```
script/cibuild
```

This will run all the rspec-puppet tests followed by puppet-lint to catch any
style issues.
1 change: 0 additions & 1 deletion Gemfile
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ source 'https://rubygems.org'

puppetversion = ENV.key?('PUPPET_VERSION') ? "= #{ENV['PUPPET_VERSION']}" : ['>= 3.2']

gem 'rake'
gem 'puppet-lint'
gem 'rspec-puppet'
gem 'puppetlabs_spec_helper'
Expand Down
19 changes: 11 additions & 8 deletions Gemfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -2,21 +2,25 @@ GEM
remote: https://rubygems.org/
specs:
diff-lcs (1.2.4)
facter (1.6.18)
hiera (1.0.0)
facter (1.7.1)
hiera (1.2.1)
json_pure
json_pure (1.8.0)
metaclass (0.0.1)
mocha (0.14.0)
metaclass (~> 0.0.1)
puppet (3.0.2)
facter (~> 1.6.11)
hiera (~> 1.0.0)
puppet (3.2.1)
facter (~> 1.6)
hiera (~> 1.0)
rgen (~> 0.6)
puppet-lint (0.3.2)
puppetlabs_spec_helper (0.4.1)
mocha (>= 0.10.5)
rake
rspec (>= 2.9.0)
rspec-puppet (>= 0.1.1)
rake (0.9.2.2)
rake (10.0.4)
rgen (0.6.2)
rspec (2.13.0)
rspec-core (~> 2.13.0)
rspec-expectations (~> 2.13.0)
Expand All @@ -32,8 +36,7 @@ PLATFORMS
ruby

DEPENDENCIES
puppet (= 3.0.2)
puppet (>= 3.2)
puppet-lint
puppetlabs_spec_helper
rake
rspec-puppet
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ lastaction - A command String that should be execute by /bin/sh once
rotate - The Integer number of rotated log files to keep on disk
(optional).
rotate_every - How often the log files should be rotated as a String.
Valid values are 'day', 'week', 'month' and 'year'
Valid values are 'hour', 'day', 'week', 'month' and 'year'
(optional). Please note, older versions of logrotate do not
support yearly log rotation.
size - The String size a log file has to reach before it will be
Expand Down
17 changes: 0 additions & 17 deletions Rakefile

This file was deleted.

3 changes: 3 additions & 0 deletions script/bootstrap
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
#!/usr/bin/env sh

bundle install --path vendor/gems --local
9 changes: 9 additions & 0 deletions script/cibuild
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
#!/usr/bin/env sh

set -e

[ -z "${PUPPET_VERSION}" ] && script/bootstrap

bundle exec rspec

bundle exec puppet-lint --with-filename --fail-on-warnings manifests
Binary file removed vendor/cache/facter-1.6.18.gem
Binary file not shown.
Binary file added vendor/cache/facter-1.7.1.gem
Binary file not shown.
Binary file removed vendor/cache/hiera-1.0.0.gem
Binary file not shown.
Binary file added vendor/cache/hiera-1.2.1.gem
Binary file not shown.
Binary file added vendor/cache/json_pure-1.8.0.gem
Binary file not shown.
Binary file removed vendor/cache/puppet-3.0.2.gem
Binary file not shown.
Binary file added vendor/cache/puppet-3.2.1.gem
Binary file not shown.
Binary file removed vendor/cache/rake-0.9.2.2.gem
Binary file not shown.
Binary file added vendor/cache/rake-10.0.4.gem
Binary file not shown.
Binary file added vendor/cache/rgen-0.6.2.gem
Binary file not shown.

0 comments on commit 365f142

Please sign in to comment.