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

Add global CONTRIBUTING.md #1

Merged
merged 1 commit into from
Sep 13, 2024
Merged

Add global CONTRIBUTING.md #1

merged 1 commit into from
Sep 13, 2024

Conversation

bastelfreak
Copy link
Member

This will allow us to purge it from modulesync_config.

@bastelfreak bastelfreak self-assigned this Jan 27, 2022
@bastelfreak bastelfreak requested a review from ekohl January 27, 2022 14:45
Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

I like this a lot, but one thing to consider here is that this is global. So non-Puppet modules will also show it. So that should be reflected.

@bastelfreak
Copy link
Member Author

I marked the development section as puppet specific, but I don't know yet how we should phrase the rest. Our other projects don't have a unified workflow. maybe something like 'check the readme.md for individual documentation and the used github actions' but that's not really helping people I think.

.github/CONTRIBUTING.md Outdated Show resolved Hide resolved
.github/CONTRIBUTING.md Outdated Show resolved Hide resolved
### Syntax and style

The test suite will run [Puppet Lint](http://puppet-lint.com/) and
[Puppet Syntax](https://github.com/gds-operations/puppet-syntax) to
Copy link
Member

Choose a reason for hiding this comment

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

This was moved to VP so the URL is out of date.

.github/CONTRIBUTING.md Outdated Show resolved Hide resolved
.github/CONTRIBUTING.md Outdated Show resolved Hide resolved
you must set an environment variable such as:

```sh
export PUPPET_VERSION="~> 5.5.6"
Copy link
Member

Choose a reason for hiding this comment

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

This will change to PUPPET_GEM_VERSION

.github/CONTRIBUTING.md Outdated Show resolved Hide resolved
.github/CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
@bastelfreak bastelfreak force-pushed the CONTRIBUTING branch 3 times, most recently from 6ef6767 to 48918d1 Compare July 5, 2024 19:47
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
bastelfreak added a commit to bastelfreak/voxpupuli-test that referenced this pull request Sep 6, 2024
bastelfreak added a commit to bastelfreak/voxpupuli-test that referenced this pull request Sep 6, 2024
bastelfreak added a commit to bastelfreak/voxpupuli-test that referenced this pull request Sep 6, 2024
@bastelfreak bastelfreak force-pushed the CONTRIBUTING branch 3 times, most recently from 2f677af to 1137802 Compare September 6, 2024 11:06
This will allow us to purge it from modulesync_config.
@bastelfreak bastelfreak merged commit 7a8d155 into master Sep 13, 2024
@bastelfreak bastelfreak deleted the CONTRIBUTING branch September 13, 2024 10:37
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.

5 participants