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

Create CONTRIBUTING.md #33

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

Create CONTRIBUTING.md #33

wants to merge 1 commit into from

Conversation

edwardtfn
Copy link
Collaborator

Just an initial file to resolve missing link on the Pull Request form. This file was copied from inspired by https://github.com/github/docs/blob/main/CONTRIBUTING.md

Pull Request Template for Home Assistant / Lovelace Card Repository

Overview

Please provide a brief description of the changes introduced in this pull request. Include any context that will help reviewers understand the motivation behind the changes, such as the problem you are trying to solve or the feature you are implementing.

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Documentation update
  • Refactoring
  • Other (please describe):

Checklist

Please ensure that you have completed the following tasks before submitting your pull request:

  • I have read the contributing guidelines for this project.
  • I have tested my changes against the dev branch (or another appropriate branch) and verified that they are working as expected.
  • I have updated the documentation (if applicable) to reflect my changes.
  • My changes adhere to the repository's code style guidelines.
  • My changes do not introduce any breaking changes or unexpected behaviors.

Related Issues / Pull Requests

Please include any related issues or pull requests that this change is associated with, and use the appropriate keywords (Closes, Fixes, Resolves) to automatically close the related issues when this pull request is merged.

  • Closes #issue_number
  • Fixes #issue_number
  • Resolves #issue_number

Additional Details

Please provide any additional information that you feel is relevant to this pull request, such as screenshots, code snippets, or examples.

Just an initial file to resolve missing link on the Pull Request form.
This file was ~~copied from~~ inspired by https://github.com/github/docs/blob/main/CONTRIBUTING.md
@edwardtfn edwardtfn added the norelease Do not bump the semantic version and release a build to HACS label Mar 27, 2023
@avataar avataar mentioned this pull request Apr 17, 2023
11 tasks
- [GitHub flow](https://docs.github.com/en/get-started/quickstart/github-flow)
- [Collaborating with pull requests](https://docs.github.com/en/github/collaborating-with-pull-requests)


Copy link
Collaborator

Choose a reason for hiding this comment

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

It would be nice to have a section here on building the project. Mentioning that the project is built with yarn and not npm and a list of the available scripts:

  • yarn build
  • yarn test
  • yarn lint
  • yarn dev

Copy link

@MrBartusek MrBartusek left a comment

Choose a reason for hiding this comment

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

This guide is very basic and doesn't really help with the setup of the project, which is probably a most important element of such a guide. I've made one guide for my Lovelace card here https://github.com/MrBartusek/MeteoalarmCard/blob/master/CONTRIBUTING.md you can take inspiration from that

@ThomDietrich ThomDietrich mentioned this pull request Apr 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
norelease Do not bump the semantic version and release a build to HACS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants