Skip to content

Sync from inner source [auto]#70

Open
well-architected-sync-bot[bot] wants to merge 5 commits intomainfrom
reposync-25209360234
Open

Sync from inner source [auto]#70
well-architected-sync-bot[bot] wants to merge 5 commits intomainfrom
reposync-25209360234

Conversation

@well-architected-sync-bot
Copy link
Copy Markdown
Contributor

Sync Contributions from the GitHub Inner Source Community

This PR merges the deployed contributions from GitHub inner source.

Source Details

  • Original Repository: github/github-well-architected-internal
  • Original SHA: c10f089f0c1b8c2bba17327327ddba6414c39883

Next Steps

  1. This PR will be merged automatically after 30 seconds.

Source Repository: github/github-well-architected-internal

Source Branch: main

Source SHA: c10f089f0c1b8c2bba17327327ddba6414c39883
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 1, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Copilot AI requested a review from KittyChiu May 1, 2026 12:27
Copy link
Copy Markdown
Contributor

Copilot AI commented May 1, 2026

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/sass/dart-sass/releases/latest
    • Triggering command: /usr/bin/curl curl -s REDACTED (http block)

If you need me to access, download, or install something from one of these locations, you can either:

@KittyChiu
Copy link
Copy Markdown
Contributor

@copilot revert the changes in 04184c3

Agent-Logs-Url: https://github.com/github/github-well-architected/sessions/e9ab14a9-bf6e-4782-b68e-4639917979af

Co-authored-by: KittyChiu <42864823+KittyChiu@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI commented May 1, 2026

@copilot revert the changes in 04184c3

Reverted in ac13f05.

KittyChiu
KittyChiu previously approved these changes May 1, 2026
@well-architected-oss-bot well-architected-oss-bot Bot enabled auto-merge (squash) May 1, 2026 12:33
@well-architected-oss-bot well-architected-oss-bot Bot deployed to staging May 1, 2026 12:33 Active
@well-architected-sync-bot
Copy link
Copy Markdown
Contributor Author

✅ Site Deployment Request Submitted

A request for live site deployment has been submitted.

Next Steps

  1. GitHub site maintainers will review your request shortly.
  2. Once approved, the changes will be deployed to the GitHub Well-Architected live site.
  3. This PR will be merged once the deployment is successful.

Note: If this contribution involves collaboration with GitHub, please contact your GitHub representative to help expedite the deployment process.


Internal xref:

colossus9
colossus9 previously approved these changes May 1, 2026
jennysharps
jennysharps previously approved these changes May 1, 2026
Copy link
Copy Markdown

@jennysharps jennysharps left a comment

Choose a reason for hiding this comment

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

CONTRIBUTING.md looks fine to me

whsalazar
whsalazar previously approved these changes May 1, 2026
Copy link
Copy Markdown

@whsalazar whsalazar left a comment

Choose a reason for hiding this comment

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

LGTM. Added section on device compliant enforcement

auto-merge was automatically disabled May 1, 2026 15:50

Head branch was pushed to by a user without write access

@whsalazar
Copy link
Copy Markdown

Added fix for PR check workflow as it was failing the test job

Copy link
Copy Markdown

@whsalazar whsalazar left a comment

Choose a reason for hiding this comment

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

Changes looks good.

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

Labels

ready for review reposync PR created from repo sync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants