Skip to content
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.

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

feat/commander #81

Merged
merged 6 commits into from
Sep 19, 2023
Merged

feat/commander #81

merged 6 commits into from
Sep 19, 2023

Conversation

jharvey10
Copy link
Contributor

@jharvey10 jharvey10 commented Sep 14, 2023

Closes #72
Closes #71
Closes #59
Closes #58
Closes #38

  • Add commander for CLI invocation
  • Add logic to invoke scopes based on config file
  • Start adding anonymization logic

@jharvey10 jharvey10 requested a review from a team as a code owner September 14, 2023 04:55
@jharvey10 jharvey10 requested review from francinelucca and removed request for a team September 14, 2023 04:55
Copy link
Contributor

@francinelucca francinelucca left a comment

Choose a reason for hiding this comment

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

looks great! just a few small Qs

src/main/core/anonymize.ts Show resolved Hide resolved
src/main/telemetry-collector.ts Outdated Show resolved Hide resolved
jharvey10 and others added 2 commits September 19, 2023 16:56
Co-authored-by: Francine Lucca <40550942+francinelucca@users.noreply.github.com>
@jharvey10 jharvey10 merged commit 3042616 into main Sep 19, 2023
3 checks passed
@jharvey10 jharvey10 deleted the feat/commander branch September 19, 2023 21:13
This was referenced Sep 21, 2023
francinelucca pushed a commit that referenced this pull request Sep 22, 2023
* Add commander for CLI invocation
* Add logic to invoke scopes based on config file
* Start adding anonymization logic
francinelucca added a commit that referenced this pull request Oct 4, 2023
* wip: wip

* wip: wip

* feat(core): first pass at full end-to-end invocation

assorted bug fixes and refactors as well

* fix: validate presence of dependency key in npm scope

* test: add edge case to getProjectRoot test

* wip: wip

* fix: rebase

* feat(core): first pass at full end-to-end invocation

assorted bug fixes and refactors as well

* fix: validate presence of dependency key in npm scope

* test: add edge case to getProjectRoot test

* feat: end-to-end run logic (feat/commander) (#81)

* Add commander for CLI invocation
* Add logic to invoke scopes based on config file
* Start adding anonymization logic

* ci: remove token from release-please.yml

* fix: replace carbon-design-system org with ibm-telemetrics (#86)

* Update CODEOWNERS

* fix: replace carbon-design-system org with ibm-telemetrics

* fix: replace carbon-design-system org with ibm-telemetrics

---------

Co-authored-by: Joe Harvey <51208233+jdharvey-ibm@users.noreply.github.com>

* release(main): release 0.2.0 (#88)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* fix(deps): update node module minor versions

* fix(deps): upgrade dependency eslint-plugin-vitest to ^0.3.1 (#79)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* fix: rename empty scope exception

* Update src/main/exceptions/empty-scope.error.ts

Co-authored-by: Joe Harvey <51208233+jdharvey-ibm@users.noreply.github.com>

---------

Co-authored-by: Joseph D. Harvey <jdharvey@us.ibm.com>
Co-authored-by: Joe Harvey <51208233+jdharvey-ibm@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants