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

refactor(CreateConfig): use correct type for validateConfigContent #201

Conversation

lukas-mertens
Copy link
Collaborator

The result of yaml.load is of type "unknown", not "object", so we can't
assume the input to validateConfigContent is even an object.

Noticed while fixing all errors caused by executing vue-tsc

Signed-off-by: Lukas Mertens [email protected]

@lukas-mertens lukas-mertens marked this pull request as ready for review October 12, 2024 10:46
@lukas-mertens lukas-mertens force-pushed the 10-12-chore_deps_install_missing_types_js-yaml branch from 4962c36 to 01cb49b Compare October 12, 2024 10:47
@lukas-mertens lukas-mertens force-pushed the 10-12-refactor_createconfig_use_correct_type_for_validateconfigcontent branch from 1b6c545 to ecbba68 Compare October 12, 2024 10:47
@lukas-mertens lukas-mertens force-pushed the 10-12-chore_deps_install_missing_types_js-yaml branch from 01cb49b to 9ca508e Compare October 12, 2024 10:52
@lukas-mertens lukas-mertens force-pushed the 10-12-refactor_createconfig_use_correct_type_for_validateconfigcontent branch from ecbba68 to 8ddb7aa Compare October 12, 2024 10:52
@lukas-mertens lukas-mertens force-pushed the 10-12-chore_deps_install_missing_types_js-yaml branch from 9ca508e to bee3b1f Compare October 12, 2024 10:54
@lukas-mertens lukas-mertens force-pushed the 10-12-refactor_createconfig_use_correct_type_for_validateconfigcontent branch from 8ddb7aa to f83adc1 Compare October 12, 2024 10:54
@lukas-mertens lukas-mertens force-pushed the 10-12-chore_deps_install_missing_types_js-yaml branch from bee3b1f to e47bf93 Compare October 12, 2024 10:57
@lukas-mertens lukas-mertens force-pushed the 10-12-refactor_createconfig_use_correct_type_for_validateconfigcontent branch from f83adc1 to 1c5ac39 Compare October 12, 2024 10:58
@lukas-mertens lukas-mertens force-pushed the 10-12-chore_deps_install_missing_types_js-yaml branch from e47bf93 to debabdf Compare October 12, 2024 11:29
@lukas-mertens lukas-mertens force-pushed the 10-12-refactor_createconfig_use_correct_type_for_validateconfigcontent branch from 1c5ac39 to 8ce3a52 Compare October 12, 2024 11:29
@lukas-mertens lukas-mertens force-pushed the 10-12-chore_deps_install_missing_types_js-yaml branch from debabdf to 2516952 Compare October 12, 2024 12:18
@lukas-mertens lukas-mertens force-pushed the 10-12-refactor_createconfig_use_correct_type_for_validateconfigcontent branch from 8ce3a52 to 316f8bf Compare October 12, 2024 12:19
Copy link

github-actions bot commented Oct 12, 2024

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 0.14% 18 / 12204
🔵 Statements 0.14% 18 / 12204
🔵 Functions 2.5% 1 / 40
🔵 Branches 9.3% 4 / 43
File Coverage
File Stmts % Branch % Funcs % Lines Uncovered Lines
Changed Files
src/components/CreateConfig.vue 0% 0% 0% 0%
Generated in workflow #440 for commit 8705f1b by the Vitest Coverage Report Action

@lukas-mertens lukas-mertens force-pushed the 10-12-chore_deps_install_missing_types_js-yaml branch from 2516952 to 00097c1 Compare October 14, 2024 13:18
@lukas-mertens lukas-mertens force-pushed the 10-12-refactor_createconfig_use_correct_type_for_validateconfigcontent branch from 316f8bf to b543959 Compare October 14, 2024 13:19
@lukas-mertens lukas-mertens changed the base branch from 10-12-chore_deps_install_missing_types_js-yaml to main October 14, 2024 13:24
The result of yaml.load is of type "unknown", not "object", so we can't
assume the input to validateConfigContent is even an object.

Noticed while fixing all errors caused by executing vue-tsc

Signed-off-by: Lukas Mertens <[email protected]>
@lukas-mertens lukas-mertens force-pushed the 10-12-refactor_createconfig_use_correct_type_for_validateconfigcontent branch from b543959 to 8705f1b Compare October 14, 2024 13:24
@lukas-mertens lukas-mertens merged commit db83351 into main Oct 15, 2024
10 checks passed
Copy link
Collaborator Author

Merge activity

  • Oct 15, 4:17 AM EDT: A user merged this pull request with Graphite.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants