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

Winter Holiday Issue Triaging and PR Review #3186

Closed
rwaskiewicz opened this issue Dec 14, 2021 · 0 comments · Fixed by #3199
Closed

Winter Holiday Issue Triaging and PR Review #3186

rwaskiewicz opened this issue Dec 14, 2021 · 0 comments · Fixed by #3199

Comments

@rwaskiewicz
Copy link
Contributor

rwaskiewicz commented Dec 14, 2021

Winter Holiday Triage and PR Review


2021.01.04 Update

Welcome back everyone! The repo's bot will no longer be telling folks that the Stencil team is on holiday. I'm going to leave this issue pinned for a bit while we work through triaging the issues that came in through during the break. Thank you for you patience ahead of time while we dig in!


With the winter holidays quickly approaching, much of the Stencil Team will soon be taking time off. During this time, issue triaging and PR review will be delayed until the team begins to return. After this period, we will work to ensure that all new issues are properly triaged and that new PRs are reviewed.

Filing an issue? Here are some things you can do to ensure that it is triaged correctly:

  • When filing an issue, fill out the template completely. This includes a description of the bug, steps to reproduce, and the version of Stencil you're using
  • Be sure to provide a code reproduction. The best way to do this is to reproduce the issue you are experiencing in a Stencil component starter library and include a link to the GitHub repo in your issue (npm init stencil -> choose 'component'). Code snippets typically are not enough to reproduce the issue, so a GitHub repo is best.
  • Be sure to follow all the steps in our Creating an Issue Guide.

Opening a PR? Here are some things you can do to ensure that it is easy to review:

  • Verify that all tests on the PR are passing. If you see a red "X" on a test, click on the "Details" link to find out why a particular test is failing.
    • If CI won't run because running the workflow hasn't been approved, you can run the unit tests and e2e tests locally (npm t and npm run test.karma.prod, respectively)
  • Include a link to the original issue in your PR description. If there is no issue yet, please create one! PRs without associated issues are not likely to get merged.
  • Be sure to follow all the steps in our Creating a Pull Request Guide.

If you are in need of immediate assistance, we recommend creating a new post on the Ionic Forums.

We have a lot of exciting things planned for Stencil in 2022, primarily focusing on open source & addressing the needs of the community. Until then, have a happy and healthy holiday, and we will see you in the new year!

The Stencil Team

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 a pull request may close this issue.

1 participant