-
Notifications
You must be signed in to change notification settings - Fork 2
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
YNGJ-1058: Create Aether Observatory #297
Conversation
A change to Portal catalog files was detected in your PR. Please visit this link to preview changes: https://portal-staging.powerapp.cloud/catalog?filters[kind]=all&filters[user]=all&filters[namespaceFilter]=yngj-1058-aether-observatory |
A change to documentation files was detected in your PR. Please visit this link to preview changes: https://portal-staging.powerapp.cloud/docs?filters[kind]=all&filters[user]=all&filters[namespaceFilter]=yngj-1058-aether-observatory |
@that-jill and @xjunior I think I've got Appraisal set up correctly. Should it be testing all packages on all pushes? I tagged a version as v0.0.1pre4 that finally made it to rubygems. |
Use public interface rather than reaching into ActiveSupport::Notifications
6c3b831
to
b261766
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works for me. @xjunior ?
Extracting
AetherObservatory
fromTalkbox
. Internal PubSub system base uponActiveSupport::Notifications