Skip to content

Commit

Permalink
Put comments on new line to make healthscore succeed
Browse files Browse the repository at this point in the history
  • Loading branch information
WilliamBergamin committed Aug 2, 2024
1 parent f95f393 commit 613eb57
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 2 additions & 1 deletion src/bundler/esbuild_bundler.ts
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,8 @@ export const EsbuildBundler = {
const result = await esbuild.build({
entryPoints: [options.entrypoint],
platform: "browser",
target: "deno1", // TODO: the versions should come from the user defined input
// TODO: the versions should come from the user defined input

Check warning on line 19 in src/bundler/esbuild_bundler.ts

View workflow job for this annotation

GitHub Actions / Health Score

src/bundler/esbuild_bundler.ts#L19

Problematic comment ("TODO") identified
target: "deno1",
format: "esm", // esm format stands for "ECMAScript module"
bundle: true, // inline any imported dependencies into the file itself
absWorkingDir: options.absWorkingDir,
Expand Down
2 changes: 1 addition & 1 deletion src/install_update.ts
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ export const updateDependencies = async () => {
// project as a means to cache the changes
if (!updateResp.error) {
try {
// TODO :: This try/catch should be nested within createUpdateResp
// TODO: This try/catch should be nested within createUpdateResp

Check warning on line 45 in src/install_update.ts

View workflow job for this annotation

GitHub Actions / Health Score

src/install_update.ts#L45

Problematic comment ("TODO") identified
// but doing so surfaces an issue with the --allow-run flag not
// being used, despite its presence and success at this level
runBuildHook();
Expand Down

0 comments on commit 613eb57

Please sign in to comment.