Skip to content
This repository has been archived by the owner on Aug 7, 2024. It is now read-only.

INT-3469 Add Questions, Workflows, Misc QoL updates #166 #68

Merged
merged 4 commits into from
May 4, 2022

Conversation

zemberdotnet
Copy link
Contributor

Description

This PR updates workflows in graph-slack, adds the codeql-analysis workflow, and questions workflow. It adds managed questions to the public repo.

Other miscellaneous or formatting changes may be included.

@zemberdotnet zemberdotnet requested a review from a team as a code owner May 2, 2022 17:07
@zemberdotnet zemberdotnet changed the title add questions, add codeql, chng package.json INT-3469 Add Questions, Workflows, Misc QoL updates #166 May 2, 2022
title: How many users are in each of my Slack channels?
description: Returns a list of slack channels and the number of users they have.
queries:
query: |
Copy link
Contributor

@VDubber VDubber May 4, 2022

Choose a reason for hiding this comment

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

I have seen this start with a - for lists. Is without acceptable if there is only one in the list?
image

Copy link
Contributor Author

@zemberdotnet zemberdotnet May 4, 2022

Choose a reason for hiding this comment

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

I think you're right. It does need -. I've just been pulling them out directly without running the validation. I expected it to catch it in the PR, and for some of the PRs it does. In other PRs like this one, it doesn't run the new flow until the next PR. I expect we will run into this a few more times, so I'll try to do some additional validation!

@@ -1,3 +1,5 @@
dist
coverage/
Copy link
Contributor

Choose a reason for hiding this comment

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

❤️

VDubber
VDubber previously approved these changes May 4, 2022
@zemberdotnet zemberdotnet force-pushed the add-questions-to-graph branch from f2b6b89 to 862210d Compare May 4, 2022 16:14
Copy link
Contributor

@VDubber VDubber left a comment

Choose a reason for hiding this comment

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

Awesome!

@zemberdotnet zemberdotnet merged commit 0c20b40 into main May 4, 2022
@zemberdotnet zemberdotnet deleted the add-questions-to-graph branch May 4, 2022 16:20
@j1-internal-automation
Copy link
Collaborator

🚀 PR was released in v5.0.0 🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants