Skip to content

docs: clarify action does not check "main" as branch during push-deploys#236

Merged
sidvishnoi merged 2 commits into
mainfrom
pchampin-patch-1
Jul 10, 2026
Merged

docs: clarify action does not check "main" as branch during push-deploys#236
sidvishnoi merged 2 commits into
mainfrom
pchampin-patch-1

Conversation

@pchampin

@pchampin pchampin commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

This change clarifies that spec-prod does not check the branch before it deploys the files. This must be done in the parameters of the GitHub action.


Preview | Diff

This change clarifies that spec-prod *does not* check the branch before it deploys the files. This must be done in the parameters of the GitHub action.
@sidvishnoi sidvishnoi changed the title Update README.md docs: clarify action does not check "main" as branch during push-deploys Jul 1, 2026

@sidvishnoi sidvishnoi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks @pchampin!

Can you also updated docs/examples.md with same note?

## Deploy to W3C using Echidna

@sidvishnoi

Copy link
Copy Markdown
Member

Filed a feature request/enhancement in #237.

@pchampin

Copy link
Copy Markdown
Contributor Author

Can you also updated docs/examples.md with same note?

done in b8a8ad0

@sidvishnoi sidvishnoi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!

@sidvishnoi sidvishnoi merged commit e9d2612 into main Jul 10, 2026
2 checks passed
@sidvishnoi sidvishnoi deleted the pchampin-patch-1 branch July 10, 2026 10:16
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