Skip to content

Conversation

@adityamandaleeka
Copy link
Member

Attempting to enable CodeQL in our CI pipeline.

@twsouthwick @dougbu @Tratcher

Attempting to enable CodeQL in our CI pipeline.
@adityamandaleeka
Copy link
Member Author

Also @wtgodbe

@twsouthwick
Copy link
Member

How do we verify that it works? I don't see anything in the builds about CodeQL - usually I've seen it as a separate check here

@adityamandaleeka
Copy link
Member Author

I set it to only run on internal builds, so I wouldn't expect to see anything new here.

Copy link
Member

@Tratcher Tratcher left a comment

Choose a reason for hiding this comment

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

That's it? 😆

YARP was a lot different
dotnet/yarp@4770410

@adityamandaleeka
Copy link
Member Author

@Tratcher I followed a general set of instructions that was sent to a bunch of teams (and therefore isn't at all tailored for our infrastructure/Arcade)... what @dougbu did in YARP is probably closer to what we should be doing here.

@dougbu
Copy link

dougbu commented Dec 7, 2022

I also wanted to be as consistent as possible across the branches. Tasks are auto injected (when you let them be) only in the repository's default branch.

@adityamandaleeka adityamandaleeka merged commit ad10f5c into main Dec 7, 2022
@adityamandaleeka adityamandaleeka deleted the codeql branch December 7, 2022 21:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants