Skip to content

Remove combo cases from active signals #770

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Nov 29, 2021
Merged

Remove combo cases from active signals #770

merged 2 commits into from
Nov 29, 2021

Conversation

krivard
Copy link
Contributor

@krivard krivard commented Nov 29, 2021

Prerequisites:

  • Unless it is a documentation hotfix it should be merged against the dev branch
  • Branch is up-to-date with the branch to be merged with, i.e. dev
  • Build is successful
  • Code is cleaned up and formatted

Summary

We stopped updating indicator-combination on November 18. This PR catches up the documentation as well as the advanced meta sheet.

@krivard krivard requested a review from capnrefsmmat November 29, 2021 21:01
capnrefsmmat
capnrefsmmat approved these changes Nov 29, 2021
View reviewed changes
Copy link
Contributor

@capnrefsmmat capnrefsmmat left a comment

Choose a reason for hiding this comment

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

looks good

@krivard krivard merged commit 9f02e5d into dev Nov 29, 2021
@krivard krivard deleted the krivard/no-combo-cases branch November 29, 2021 21:14
@krivard krivard mentioned this pull request Nov 29, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@capnrefsmmat capnrefsmmat capnrefsmmat approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@krivard @capnrefsmmat