Skip to content

docs: add note about demos on PRs #6606

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Jul 21, 2025
Merged

docs: add note about demos on PRs #6606

merged 6 commits into from
Jul 21, 2025

Conversation

Patrick-Erichsen
Copy link
Collaborator

@Patrick-Erichsen Patrick-Erichsen commented Jul 14, 2025 •
edited by cubic-dev-ai bot
Loading

Summary by cubic

Added a note to the PR template and contributing guide asking for a short screen recording when submitting new features, to make reviews easier.

@Patrick-Erichsen Patrick-Erichsen requested a review from a team as a code owner July 14, 2025 02:00
@Patrick-Erichsen Patrick-Erichsen requested review from RomneyDa and removed request for a team July 14, 2025 02:00
@netlify Netlify
Copy link

netlify bot commented Jul 14, 2025 •
edited
Loading

❌ Deploy Preview for continuedev failed. Why did it fail? →

Name Link
🔨 Latest commit 653ed97
🔍 Latest deploy log https://app.netlify.com/projects/continuedev/deploys/68747efb23cab8000892b167

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jul 14, 2025
cubic-dev-ai[bot]
cubic-dev-ai bot reviewed Jul 14, 2025
View reviewed changes
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

cubic reviewed 2 files and found no issues. Review PR in cubic.dev.

@github-actions GitHub Actions
Copy link

⚠️ Conventional Commit Format

Your commit messages don't follow the conventional commit format, but this won't block your PR from being merged. We recommend downloading this extension if you are using VS Code.

Expected Format:

<type>[optional scope]: <description>

[optional body]

[optional footer(s)]

Examples:

  • feat: add changelog generation support
  • fix: resolve login redirect issue
  • docs: update README with new instructions
  • chore: update dependencies

Valid Types:

feat, fix, docs, style, refactor, perf, test, build, ci, chore, revert

This helps with:

  • 📝 Automatic changelog generation
  • 🚀 Automated semantic versioning
  • 📊 Better project history tracking

This is a non-blocking warning - your PR can still be merged without fixing this.

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Jul 17, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 17, 2025
@RomneyDa
Copy link
Collaborator

@Patrick-Erichsen looks like failing several tests

@RomneyDa RomneyDa merged commit d4983da into main Jul 21, 2025
76 of 78 checks passed
@RomneyDa RomneyDa deleted the pe/contributing-docs-update branch July 21, 2025 18:37
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Jul 21, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Jul 21, 2025
@sestinj
Copy link
Contributor

sestinj commented Jul 22, 2025

🎉 This PR is included in version 1.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@cubic-dev-ai cubic-dev-ai[bot] cubic-dev-ai[bot] left review comments

@RomneyDa RomneyDa RomneyDa approved these changes

Assignees
No one assigned
Labels
lgtm This PR has been approved by a maintainer released size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
Issues and PRs
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Patrick-Erichsen @RomneyDa @sestinj