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

chore: add contributing guide #18

Merged
merged 2 commits into from
Nov 22, 2023
Merged

Conversation

juliodialpad
Copy link
Collaborator

Description

Added general contributing guide with links to specific contributing guides.
This will help us while we decide what else we want to add to general contributing guide.

Pull Request Checklist

  • Ask the contributors if a similar effort is already in process or has been solved.
  • Review the contribution guidelines.
  • Use staging as your pull request's base branch. (All PRs using production as its base branch will be declined).
  • Ensure all gulp scripts successfully compile.
  • Update, remove, or extend all affected documentation.
  • Ensure no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).

Obligatory GIF (super important!)

Copy link

✔️ Deploy Preview ready!
😎 Browse the preview: https://dialpad.design/deploy-previews/pr-18
🔨 If you experience an SSL issue then wait 2 minutes and try again.

Comment on lines 12 to 13
## Dialtone Vue 3
[CONTRIBUTING.md](../packages/dialtone-vue3/.github/CONTRIBUTING.md)
Copy link
Contributor

Choose a reason for hiding this comment

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

what is packages/dialtone-vue3?? this doesn't exist

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oh sorry, I have it on my local because I'm working on that, will remove it for now.

@braddialpad
Copy link
Contributor

Approved after you make the change.

Copy link

✔️ Deploy Preview ready!
😎 Browse the preview: https://dialpad.design/deploy-previews/pr-18
🔨 If you experience an SSL issue then wait 2 minutes and try again.

@juliodialpad juliodialpad merged commit 4767bde into staging Nov 22, 2023
2 checks passed
@juliodialpad juliodialpad deleted the chore/add-contributing-guide branch November 22, 2023 21:02
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