Meta tags:
description=This PR separates the core cpp packs into codeql/cpp-queries and codeql/cpp-all. There are very few lines of code changed. Almost all changes are moving files around.;

Headings (most frequently used words):

commented, aug, 17, 2021, comment, aeisenberg, packaging, refactor, cpp, libraries, 6488, footer, dbartol, left, choose, reason, for, hiding, this, conversation, rdmarsh2, adityasharad, navigation,

Text of the page (most frequently used words):
this (30), the (27), 2021 (24), all (21), #github (20), cpp (20), #aeisenberg (18), aug (15), code (12), changes (11), dbartol (11), comment (11), and (11), applied (10), from (10), pack (10), sign (9), for (9), august (9), pull (8), codeql (8), you (7), suggestions (7), request (7), merged (7), adityasharad (7), sorry (7), something (7), went (7), wrong (7), reactions (7), copy (7), link (7), packaging (7), can (6), that (6), cannot (6), suggestion (6), change (6), have (6), branch (6), into (6), are (6), reason (6), contributor (6), jump (6), internal (5), files (5), checks (5), main (5), libraries (5), commented (5), review (5), with (4), about (4), security (4), these (4), issues (4), note (4), semmle (4), account (4), choose (4), more (4), hiding (4), there (4), compare (4), force (4), pushed (4), refactor (4), actions (4), open (4), 6488 (4), your (3), 2023 (3), while (3), line (3), resolved (3), outdated (3), lines (3), not (3), batch (3), commit (3), only (3), projects (3), documentation (3), should (3), approved (3), conversation (3), passed (3), view (3), hide (3), moving (3), around (3), team (3), learn (3), was (3), 03d6b15 (3), added (3), uses (3), fix (3), packs (3), changed (3), characters (3), unicode (3), commits (3), discussions (3), repository (3), signed (2), another (2), tab (2), window (2), refresh (2), session (2), reload (2), blog (2), pricing (2), contact (2), privacy (2), terms (2), footer (2), navigation (2), merge (2), deleted (2), add (2), single (2), closed (2), were (2), none (2), yet (2), merging (2), may (2), milestone (2), python (2), does (2), need (2), required (2), java (2), sync (2), depends (2), labels (2), already (2), free (2), details (2), fail (2), removed (2), lgtm (2), duplicate (2), off (2), topic (2), abuse (2), spam (2), will (2), displayed (2), describe (2), others (2), problem (2), left (2), make (2), author (2), label (2), lib (2), 3b05a60 (2), references (2), query (2), separates (2), core (2), queries (2), very (2), few (2), almost (2), csharp (2), new (2), hidden (2), file (2), bidirectional (2), issue (2), community (2), project (2), insights (2), requests (2), source (2), manage (2), packages (2), web (2), capture (2), mar (2), out, perform, action, time, training, api, docs, status, inc, queued, multi, comments, pending, reviews, has, been, marked, must, existing, order, create, valid, applying, supported, one, per, viewing, subset, invalid, because, made, participants, successfully, close, development, assignees, reviewers, join, drop, redundant, columns, relations, 6477, folders, mentioned, geoffw0, 21d03cd, except, known, bug, when, using, non, branches, expected, since, tests, corresponding, qldoc, collaborator, still, 6982ff8, dismissed, via, stale, yes, trying, smallest, possible, first, complete, later, self, assigned, once, languages, refactored, probably, sweep, through, whole, repo, ensure, old, names, etc, gone, well, librarypathdependencies, previously, also, some, library, path, dependencies, 88ceb42, migrate, experimental, folder, d13ce88, recursive, d8e4e25, 520af02, 2c5dd2d, thanks, right, let, remove, them, looks, like, handful, under, frameworks, system, data, accidentally, committed, rdmarsh2, 9779426, 5e2e014, bot, requested, owners, style, naming, scheme, e566fb9, update, suite, helpers, qlpack, show, contains, text, interpreted, compiled, differently, than, what, appears, below, editor, reveals, bottom, clicking, agree, our, occasionally, send, related, emails, statement, service, password, email, address, pick, username, its, maintainers, question, 264, 585, star, fork, notifications, public, message, organization, suggested, results, collections, trending, topics, repositories, articles, readme, fund, developers, sponsors, resources, customer, stories, case, studies, devsecops, devops, automation, solution, education, startups, teams, enterprise, solutions, skills, features, explore, collaborate, outside, plan, track, work, write, better, copilot, instant, dev, environments, codespaces, find, vulnerabilities, host, automate, any, workflow, product, toggle, skip, content, wayback, machine, http, archive, org, 20230306174704, https, com, timestamps, success, 2024, 2022, apr, feb,

Text of the page (random words):
packaging refactor cpp libraries by aeisenberg pull request 6488 github codeql github 1 capture 06 mar 2023 feb mar apr 06 2022 2023 2024 success fail about this capture timestamps the wayback machine http web archive org web 20230306174704 https github com github codeql pull 6488 skip to content toggle navigation sign up product actions automate any workflow packages host and manage packages security find and fix vulnerabilities codespaces instant dev environments copilot write better code with ai code review manage code changes issues plan and track work discussions collaborate outside of code explore all features documentation github skills blog solutions for enterprise teams startups education by solution ci cd automation devops devsecops case studies customer stories resources open source github sponsors fund open source developers the readme project github community articles repositories topics trending collections pricing in this repository all github jump to no suggested jump to results in this repository all github jump to in this organization all github jump to in this repository all github jump to sign in sign up message github codeql public notifications fork 1 3k star 5 8k code issues 585 pull requests 264 discussions actions projects 0 security insights more code issues pull requests discussions actions projects security insights 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 pick a username email address password 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 packaging refactor cpp libraries 6488 merged adityasharad merged 5 commits into main from aeisenberg pack cpp aug 17 2021 merged packaging refactor cpp libraries 6488 adityasharad merged 5 commits into main from aeisenberg pack cpp aug 17 2021 conversation 6 commits 5 checks 3 files changed conversation this file contains 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 show hidden characters copy link contributor aeisenberg commented aug 17 2021 this pr separates the core cpp packs into codeql cpp queries and codeql cpp all there are very few lines of code changed almost all changes are moving files around sorry something went wrong all reactions packaging update suite helpers qlpack e566fb9 uses new style naming scheme aeisenberg requested review from a team as code owners august 17 2021 01 15 github actions bot added c c documentation java js python labels aug 17 2021 aeisenberg force pushed the aeisenberg pack cpp branch from 5e2e014 to 9779426 compare august 17 2021 03 10 copy link contributor rdmarsh2 commented aug 17 2021 it looks like a handful of files under csharp ql lib semmle code csharp frameworks system data were accidentally committed all reactions sorry something went wrong copy link contributor author aeisenberg commented aug 17 2021 thanks you re right let me remove them all reactions sorry something went wrong packaging refactor the cpp libraries 2c5dd2d this pr separates the core cpp packs into codeql cpp queries and codeql cpp all there are very few lines of code changed almost all changes are moving files around aeisenberg force pushed the aeisenberg pack cpp branch from 520af02 to 3b05a60 compare august 17 2021 18 23 packaging fix query pack references d8e4e25 we can t have recursive references to query packs aeisenberg force pushed the aeisenberg pack cpp branch from 3b05a60 to d13ce88 compare august 17 2021 20 04 dbartol added the no change note required this pr does not need a change note label aug 17 2021 packaging migrate cpp experimental semmle folder to lib 88ceb42 also fix up some library path dependencies dbartol previously approved these changes aug 17 2021 view changes copy link contributor dbartol left a comment there was a problem hiding this comment 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 hide comment lgtm once we have all languages refactored we should probably make a sweep through the whole repo to ensure that all uses of the old pack names codeql cpp etc are gone as well as all uses of librarypathdependencies sorry something went wrong all reactions dbartol added the depends on internal pr this pr should only be merged in sync with an internal semmle pr label aug 17 2021 dbartol self assigned this aug 17 2021 copy link contributor author aeisenberg commented aug 17 2021 yes i was trying to make the smallest change possible at first and we can complete the changes later all reactions sorry something went wrong merge branch main into aeisenberg pack cpp 03d6b15 aeisenberg dismissed dbartol s stale review via 03d6b15 august 17 2021 22 29 aeisenberg force pushed the aeisenberg pack cpp branch from 6982ff8 to 03d6b15 compare august 17 2021 22 29 dbartol approved these changes aug 17 2021 view changes copy link contributor dbartol left a comment there was a problem hiding this comment 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 hide comment still lgtm sorry something went wrong all reactions adityasharad removed the request for review from a team august 17 2021 23 30 adityasharad removed request for a team august 17 2021 23 30 copy link collaborator adityasharad commented aug 17 2021 all checks have passed on this pr except checks known bug when using non main branches and qldoc checks expected to fail since we are moving libraries around the tests have all passed on the corresponding internal pr merging all reactions sorry something went wrong hide details view details adityasharad merged commit 21d03cd into main aug 17 2021 16 of 18 checks passed adityasharad deleted the aeisenberg pack cpp branch august 17 2021 23 44 geoffw0 mentioned this pull request aug 18 2021 drop redundant columns from files and folders relations 6477 closed sign up for free to join this conversation on github already have an account sign in to comment reviewers dbartol dbartol approved these changes assignees dbartol labels c c depends on internal pr this pr should only be merged in sync with an internal semmle pr documentation java js no change note required this pr does not need a change note python projects none yet milestone no milestone development successfully merging this pull request may close these issues none yet 4 participants add this suggestion to a batch that can be applied as a single commit this suggestion is invalid because no changes were made to the code suggestions cannot be applied while the pull request is closed suggestions cannot be applied while viewing a subset of changes only one suggestion per line can be applied in a batch add this suggestion to a batch that can be applied as a single commit applying suggestions on deleted lines is not supported you must change the existing code in this line in order to create a valid suggestion outdated suggestions cannot be applied this suggestion has been applied or marked resolved suggestions cannot be applied from pending reviews suggestions cannot be applied on multi line comments suggestions cannot be applied while the pull request is queued to merge footer 2023 github inc footer navigation terms privacy security status docs contact github pricing api training blog about you can t perform that action at this time you signed in with another tab or window reload to refresh your session you signed out in another tab or window reload to refresh your session