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.

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

refactor(utils): prepare for core refactoring #78

Closed
BioPhoton opened this issue Oct 2, 2023 · 0 comments · Fixed by #81
Closed

refactor(utils): prepare for core refactoring #78

BioPhoton opened this issue Oct 2, 2023 · 0 comments · Fixed by #81
Milestone
1. Internal MVP

Comments

@BioPhoton
Copy link
Collaborator

This is a suggestion on how the new files should get arranged:

utils:

  • md
  • execute-process.ts
  • report-to-md.ts
  • report-to-stdout.ts
  • utils.ts

core:

  • execute-plugin.ts
  • persist.ts
@BioPhoton BioPhoton added 🐛 bug something isn't working and removed 🐛 bug something isn't working labels Oct 2, 2023
This was referenced Oct 2, 2023
Closed
Merged
BioPhoton added a commit that referenced this issue Oct 2, 2023
@BioPhoton @matejchalk
feat(utils): move parts from utils into core (#81)
dd0a180 
closes #78

---------

Co-authored-by: Matěj Chalk <34691111+matejchalk@users.noreply.github.com>
@vmasek vmasek added this to the MVP milestone Oct 6, 2023
@BioPhoton BioPhoton mentioned this issue Oct 18, 2023
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1. Internal MVP
Development

Successfully merging a pull request may close this issue.

feat(utils): move parts from utils into core code-pushup/cli
2 participants
@vmasek @BioPhoton