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: cli draft #3255

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

feat: cli draft #3255

wants to merge 1 commit into from

Conversation

H01001000
Copy link
Contributor

@H01001000 H01001000 commented Dec 3, 2023

The PR generates a message type file from the .discordenorc.json config file.
The type file is located at node_modules/.discordeno/transformer/src

@H01001000 H01001000 linked an issue Dec 3, 2023 that may be closed by this pull request
@github-actions github-actions bot added pkg-all Affects every package t-feat A new feature labels Dec 3, 2023
@H01001000 H01001000 self-assigned this Dec 3, 2023
Copy link

codecov bot commented Dec 3, 2023

Codecov Report

Attention: 199 lines in your changes are missing coverage. Please review.

Comparison is base (b567c13) 77.00% compared to head (4f7ec6a) 76.32%.
Report is 2 commits behind head on main.

Files Patch % Lines
...cordeno/src/bin/generators/messageTypeGenerator.ts 0.00% 142 Missing ⚠️
packages/discordeno/src/bin/utils/readConfig.ts 0.00% 38 Missing ⚠️
packages/discordeno/src/bin/index.ts 0.00% 14 Missing ⚠️
...kages/discordeno/src/bin/types/DiscordenoConfig.ts 0.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3255      +/-   ##
==========================================
- Coverage   77.00%   76.32%   -0.69%     
==========================================
  Files         186      189       +3     
  Lines       22108    22306     +198     
  Branches      721      721              
==========================================
  Hits        17025    17025              
- Misses       5071     5269     +198     
  Partials       12       12              
Flag Coverage Δ *Carryforward flag
bot 54.06% <ø> (ø)
bot-e2e 53.97% <ø> (ø) Carriedforward from b567c13
bot-unit 48.77% <ø> (ø)
discordeno 0.47% <0.00%> (-7.86%) ⬇️
discordeno-unit 0.47% <0.00%> (-7.86%) ⬇️
gateway 74.51% <ø> (ø)
gateway-integration 74.51% <ø> (ø)
gateway-unit 32.28% <ø> (ø)
rest 83.83% <ø> (ø)
rest-e2e 27.56% <ø> (ø) Carriedforward from b567c13
rest-unit 70.98% <ø> (ø)
types 100.00% <ø> (ø)
types-unit 100.00% <ø> (ø)
utils 89.71% <ø> (ø)
utils-unit 89.71% <ø> (ø)

*This pull request uses carry forward flags. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@@ -0,0 +1,5 @@
{
Copy link
Contributor

Choose a reason for hiding this comment

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

should this be in the repo? also i thought we wanted to go with discordeno.config.js file name?

@@ -0,0 +1,142 @@
const messageTypeProps = {
Copy link
Contributor

Choose a reason for hiding this comment

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

is this autogenerated or manually created? if autogenerated it shouldnt be in the repo, if manual, im not a fan of having this written twice here.

@@ -0,0 +1,5 @@
export interface DiscordenoConfig {
desiredProps: {
message: string[]
Copy link
Contributor

Choose a reason for hiding this comment

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

whats this for?

@Fleny113 Fleny113 mentioned this pull request Feb 22, 2024
28 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg-all Affects every package t-feat A new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CLI] npx discordeno generate
2 participants