FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add experimental `build-mode` input by henrymercer · Pull Request #2108 · github/codeql-action · GitHub

Add experimental build-mode input - #2108

Merged
henrymercer merged 7 commits into
mainfrom
henrymercer/build-mode-input
Feb 2, 2024
Merged

Add experimental build-mode input#2108
henrymercer merged 7 commits into
mainfrom
henrymercer/build-mode-input

Conversation

Copy link
Copy Markdown
Contributor

This PR introduces an experimental build-mode input to the init Action. This is currently for internal testing only.

We forward this input to codeql database init and add a test to check that the build mode is forwarded to the database as expected, and Java buildless extraction completes successfully.

This PR makes use of the config refactoring implemented in #2107.

Merge / deployment checklist

  • Confirm this change is backwards compatible with existing workflows.
  • Confirm the readme has been updated if necessary.
  • Confirm the changelog has been updated if necessary.

henrymercer requested a review from a team as a code owner January 31, 2024 14:49
henrymercer force-pushed the henrymercer/build-mode-input branch from c1e0768 to 463930c Compare January 31, 2024 14:55
Base automatically changed from henrymercer/refactor-config to main January 31, 2024 18:41
aeisenberg previously approved these changes Jan 31, 2024

aeisenberg left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I think the action iput description can be clarified, but it's not a blocking change.

Comment thread init/action.yml

aeisenberg left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM!

angelapwen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

🆒 😄

henrymercer merged commit 81eb6b2 into main Feb 2, 2024
henrymercer deleted the henrymercer/build-mode-input branch February 2, 2024 10:46
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

3 participants


Back | FazBrowse Home | New Git URL