Skip to content

Finalizing data-search script & cron job#30

Merged
Suhyma merged 12 commits intomainfrom
suhyma
Mar 8, 2023
Merged

Finalizing data-search script & cron job#30
Suhyma merged 12 commits intomainfrom
suhyma

Conversation

@Suhyma
Copy link
Copy Markdown
Collaborator

@Suhyma Suhyma commented Feb 27, 2023

Just added some code to format the final JSON after the search is completed. The properties of the JSON elements now match the output of the original search.sh script.

Refs #22

@Suhyma Suhyma requested review from jvwong and maxkfranz February 27, 2023 19:18
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 27, 2023

Codecov Report

Merging #30 (05065f9) into main (a6900d1) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##            main     #30   +/-   ##
=====================================
  Coverage   0.00%   0.00%           
=====================================
  Files          2       2           
  Lines        106     106           
=====================================
  Misses       106     106           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Comment thread src/cli.js Outdated
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Mar 2, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 05065f9
Status:🚫  Build failed.

View logs

@Suhyma Suhyma requested a review from jvwong March 2, 2023 21:00
@maxkfranz
Copy link
Copy Markdown
Member

@Suhyma @jvwong

There's a conflict in the PR that would need to be addressed in both the PR and @Suhyma's branch going forward.

Suhyma added 5 commits March 6, 2023 14:42
# Conflicts:
#	src/cli.js
# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.

Updating the upstream and downstream routes
@Suhyma Suhyma changed the title Formatting search output JSON Finalizing data-search script & cron job Mar 8, 2023
Copy link
Copy Markdown
Member

@jvwong jvwong left a comment

Choose a reason for hiding this comment

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

Great - just wondering about the file open/close.

Comment thread src/data-search.js Outdated
Comment thread src/data-search.js Outdated
Comment thread src/data-search.js Outdated
Comment thread src/data-search.js Outdated
@Suhyma Suhyma requested a review from jvwong March 8, 2023 18:46
Copy link
Copy Markdown
Member

@jvwong jvwong left a comment

Choose a reason for hiding this comment

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

Works for me.

@Suhyma Suhyma merged commit 6fcc22e into main Mar 8, 2023
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.

3 participants