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

use ATM pack v0.4.0 for CLI v2.11.3 and above by kaeluka · Pull Request #1351 · github/codeql-action · GitHub

use ATM pack v0.4.0 for CLI v2.11.3 and above - #1351

Merged
kaeluka merged 1 commit into
github:mainfrom
kaeluka:release-atm-v0.4.0
Nov 8, 2022
Merged

kaeluka merged 1 commit into
github:mainfrom
kaeluka:release-atm-v0.4.0

Conversation

kaeluka commented Nov 8, 2022
edited
Loading

Copy link
Copy Markdown

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.

kaeluka force-pushed the release-atm-v0.4.0 branch from e09bab4 to 5194090 Compare November 8, 2022 12:36

kaeluka commented Nov 8, 2022

Copy link
Copy Markdown
Author

I'm assuming that README and CHANGELOG don't need to be updated, since the last PR also didn't.

kaeluka commented Nov 8, 2022

Copy link
Copy Markdown
Author

I'm also assuming that backwards-compatibility is given, but this is my first PR to this repo, so I'm asking the reviewer (@henrymercer?) to confirm that, please :)

kaeluka marked this pull request as ready for review November 8, 2022 12:41
kaeluka requested a review from a team as a code owner November 8, 2022 12:41
kaeluka force-pushed the release-atm-v0.4.0 branch from 5194090 to eaa8869 Compare November 8, 2022 12:44
kaeluka force-pushed the release-atm-v0.4.0 branch from eaa8869 to 01c4458 Compare November 8, 2022 12:53
kaeluka enabled auto-merge November 8, 2022 13:39
kaeluka merged commit 73ff862 into github:main Nov 8, 2022

Copy link
Copy Markdown
Contributor

Perhaps it is time to start updating the changelog (separate PR is fine), at least for our own convenience. Something similar to what we do when updating the CLI version will suffice: Update the ML-powered additional query pack for JavaScript to version x.y.z.

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