Skip to content
Unverified Commit ab74d876 authored by iamjpotts's avatar iamjpotts Committed by GitHub
Browse files

chore(ci): Only dry run a release, and only publish github pages, when ci...

chore(ci): Only dry run a release, and only publish github pages, when ci workflow is triggered from main smithy-rs repository (#3663)

Similar to https://github.com/smithy-lang/smithy-rs/pull/3597

.

Only run these two workflows when they are triggered from the
`smithy-lang/smithy-rs` repository, not from any fork.

## Motivation and Context
<!--- Why is this change required? What problem does it solve? -->
<!--- If it fixes an open issue, please link to the issue here -->

The scheduled dry run fails daily in forks, failing each time,
triggering alerts to the owner of the fork repository.

The github pages publishing workflow does not run regularly in forks,
but does run when a fork is synced. It would be undesirable for an
arbitrary fork to publish.

## Description
<!--- Describe your changes in detail -->

On each of the two github workflows, add an `if` condition testing the
repository owner and name. Only run if it matches the official
repository, and is not a fork.

## Testing
<!--- Please describe in detail how you tested your changes -->
<!--- Include details of your testing environment, and the tests you ran
to -->
<!--- see how your change affects other areas of the code, etc. -->
n/a

## Checklist
<!--- If a checkbox below is not applicable, then please DELETE it
rather than leaving it unchecked -->
- [ ] I have updated `CHANGELOG.next.toml` if I made changes to the
smithy-rs codegen or runtime crates
- [ ] I have updated `CHANGELOG.next.toml` if I made changes to the AWS
SDK, generated SDK code, or SDK runtime crates

n/a in both cases
----

_By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice._

Signed-off-by: default avatarJoshua Potts <8704475+iamjpotts@users.noreply.github.com>
parent 59ad183d
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment