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

advertise a link to an experimental build, if one exists #5037

Merged
merged 1 commit into from
May 3, 2024

Conversation

emptyrivers
Copy link
Contributor

try opening a pull request (any kind!) over on my fork, or see emptyrivers#12 for an example.

Will only post links for pull requests with the feature preview label.

@emptyrivers emptyrivers added the ✨Housekeeping Chores which need doing, like travis config, dependencies, or changing data to keep up with patches. label Apr 29, 2024
@emptyrivers emptyrivers requested a review from Stanzilla April 29, 2024 19:19
on: [pull_request]
on:
pull_request:
types: [opened, reopened, synchronize, labeled]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

slight weirdness here: a build is retriggered if someone edits the labels. I wanted to keep this to just 'feature preview' prs, so there's some discernment about which changes we actually expect someone to install & give feedback on. But we could also ditch that if frivolous retriggers are considered annoying

with:
name: WeakAuras-PR#${{ github.event.number }}-${{ github.sha }}
path: .release/

- name: test
Copy link
Contributor

Choose a reason for hiding this comment

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

what does this do exactly?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

anything you want, so long as that isn't very useful

Copy link
Contributor

Choose a reason for hiding this comment

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

ConfusedConfusedFaceGIF

Copy link
Contributor

Choose a reason for hiding this comment

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

It obviously tests duh

@Boneshockx
Copy link
Contributor

What if this posted to #wow-addon and pings @WAalpha

@InfusOnWoW InfusOnWoW merged commit dbf6c7b into WeakAuras:main May 3, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Housekeeping Chores which need doing, like travis config, dependencies, or changing data to keep up with patches.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants