Skip to content

refactor: global options location - #258

Merged
BioPhoton merged 12 commits into
mainfrom
refactor-cli-typings
Nov 15, 2023
Merged

BioPhoton merged 12 commits into
mainfrom
refactor-cli-typings

Conversation

@BioPhoton

@BioPhoton BioPhoton commented Nov 15, 2023 •

Copy link
Copy Markdown
Collaborator

This PR could go before #246 as there are a couple of questions it would resolve.

It includes:

  • relocating of global options
  • separation responsibilities between cli and core
  • type improvements for commands
  • deleting empty file 👻
  • small type tweaks

Out of scope 😇:

  • 🧹 utils - merge file utils into 1 file

@BioPhoton
BioPhoton requested a review from IKatsuba November 15, 2023 01:18
@BioPhoton BioPhoton self-assigned this Nov 15, 2023
@BioPhoton
BioPhoton marked this pull request as ready for review November 15, 2023 01:52
@BioPhoton
BioPhoton requested a review from matejchalk November 15, 2023 02:48
Comment thread .gitignore
Comment thread packages/cli/src/lib/autorun/command-object.ts
Comment thread packages/core/package.json Outdated
Comment thread packages/core/src/lib/implementation/read-code-pushup-config.ts Outdated
BioPhoton and others added 3 commits November 15, 2023 11:51
Co-authored-by: Matěj Chalk <34691111+matejchalk@users.noreply.github.com>
Comment thread packages/core/src/lib/implementation/read-code-pushup-config.ts Outdated
@BioPhoton
BioPhoton requested a review from matejchalk November 15, 2023 16:15

@BioPhoton BioPhoton left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Should be ready to go

@matejchalk matejchalk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM 🙂

@BioPhoton
BioPhoton merged commit 7662cc5 into main Nov 15, 2023
@BioPhoton
BioPhoton deleted the refactor-cli-typings branch November 15, 2023 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants