Skip to content

Document that external pull requests are not accepted - #826

Merged
fpigeonjr merged 1 commit into
masterfrom
gh-contributing-policy-external-prs-not-accepted
Sep 10, 2026
Merged

Document that external pull requests are not accepted#826
fpigeonjr merged 1 commit into
masterfrom
gh-contributing-policy-external-prs-not-accepted

Conversation

@fpigeonjr

Copy link
Copy Markdown
Contributor

Description

Adds an explicit "Accepting Contributions" statement to this repository's contributing guidelines, stating that external pull requests are not currently accepted.

Motivation and Context

Confirmed with the PO in standup: this repository does not accept external contributions at this time. We currently have open PRs from outside contributors that will need to be closed as a result, so we want the policy documented up front to set expectations for anyone who finds the repo and considers opening a PR.

Type of Change

  • Documentation / configuration update

How to Test

  1. Read the diff — confirm the new section reads clearly and links to the issues page as the alternative contribution path.
  2. No build/lint/test impact; docs-only change.

The team confirmed in standup that this repository does not accept
external contributions. Add an explicit statement to CONTRIBUTING.md
(or README.md where no CONTRIBUTING.md previously existed) so
contributors are told upfront rather than discovering it after
submitting a PR, and direct them to open an issue instead.
@fpigeonjr
fpigeonjr requested a review from a team September 10, 2026 16:31
@github-actions

github-actions Bot commented Sep 10, 2026

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-09-10 19:08 UTC

@github-actions

Copy link
Copy Markdown

Component / Story Coverage Report

100% coverage (52 / 52 stories have Playwright specs) — threshold: 90%

Summary

Metric Value
Total stories 52
Covered by specs 52
Not covered 0
Coverage % 100%
Threshold 90%
Status PASS ✅

Coverage metric

For this SCSS-only library, "coverage" means every Storybook story (.stories.js)
in sam-styles/packages/ has at least one matching Playwright spec in tests/storybook/.

Uncovered stories

All stories are covered 🎉

Covered stories

Story file Status
branding/colors/colors.stories.js ✅ Covered
branding/elements/elements.stories.js ✅ Covered
branding/icons/icons.stories.js ✅ Covered
branding/typography/fields/fields.stories.js ✅ Covered
branding/typography/heading/heading.stories.js ✅ Covered
branding/typography/lists/lists.stories.js ✅ Covered
branding/typography/paragraph/paragraph.stories.js ✅ Covered
branding/typography/typesetting/typesetting.stories.js ✅ Covered
components/accordion/accordion.stories.js ✅ Covered
components/actions/actions.stories.js ✅ Covered
components/alerts/alert.stories.js ✅ Covered
components/autocomplete/autocomplete.stories.js ✅ Covered
components/button/circle/circle-button.stories.js ✅ Covered
components/button/pill/pill-button.stories.js ✅ Covered
components/button/standard/Button.stories.js ✅ Covered
components/button-group/button-grop.stories.js ✅ Covered
components/button-row/button-row.stories.js ✅ Covered
components/card/card.stories.js ✅ Covered
components/collection/collection.stories.js ✅ Covered
components/dashboard/event/event.stories.js ✅ Covered
components/dashboard/statistics/alt.stories.js ✅ Covered
components/dashboard/statistics/default.stories.js ✅ Covered
components/dashboard/tile/SDStile/sdstile.stories.js ✅ Covered
components/dashboard/tile/SDStile-outline/tileoutline.stories.js ✅ Covered
components/history/history.stories.js ✅ Covered
components/pagination/Pagination.stories.js ✅ Covered
components/process-list/process-list.stories.js ✅ Covered
components/range-slider/range-slider.stories.js ✅ Covered
components/side-nav/side-nav.stories.js ✅ Covered
components/stepindicator/stepindicator.stories.js ✅ Covered
components/summarybox/summarybox.stories.js ✅ Covered
components/tables/tables.stories.js ✅ Covered
components/tags/tags.stories.js ✅ Covered
components/toasts/toasts.stories.js ✅ Covered
form-controls/address-form/address-form.stories.js ✅ Covered
form-controls/checkbox/checkbox.stories.js ✅ Covered
form-controls/dropdown/dropdown.stories.js ✅ Covered
form-controls/input/input.stories.js ✅ Covered
form-controls/name-form/name-form.stories.js ✅ Covered
form-controls/radio/radio.stories.js ✅ Covered
form-controls/search/search.stories.js ✅ Covered
misc/pop-up/pop-up.stories.js ✅ Covered
misc/search-layout/search-layout.stories.js ✅ Covered
misc/sort/sort.stories.js ✅ Covered
misc/toolbar/toolbar.stories.js ✅ Covered
patterns/pages/pages.stories.js ✅ Covered
sds-styles/stories/sds-styles.stories.js ✅ Covered
structure/banner/banner.stories.js ✅ Covered
structure/footer/footer.stories.js ✅ Covered
structure/header/header.stories.js ✅ Covered
structure/sds-navbar/sds-navbar.stories.js ✅ Covered
structure/seal/seal.stories.js ✅ Covered

@fpigeonjr
fpigeonjr merged commit 44ce8a0 into master Sep 10, 2026
11 checks passed
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