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

build: Fix issue with workflow not correctly parsing JSON string. #96

Merged
merged 1 commit into from
Feb 2, 2025

Conversation

nasa42
Copy link
Owner

@nasa42 nasa42 commented Feb 2, 2025

According to job https://github.com/nasa42/webterm/actions/runs/13100985186/job/36549192322, following command is run:

  echo "Parsing releases from release-plz..."
  RELEASES="[{"package_name":"webterm-core","prs":[],"tag":"webterm-core-v0.2.1","version":"0.2.1"},{"package_name":"webterm-agent","prs":[],"tag":"webterm-agent-v0.2.1","version":"0.2.1"},{"package_name":"webterm-relay","prs":[],"tag":"webterm-relay-v0.2.1","version":"0.2.1"}]"
  echo "Raw releases data: $RELEASES"

  echo "$RELEASES" | jq -c '.[]' | while read release; do
    package_name=$(echo "$release" | jq -r '.package_name')
    if [[ "$package_name" == "webterm-agent" || "$package_name" == "webterm-relay" ]]; then
      echo "Found relevant release: $package_name"
      echo "$release" >> relevant_releases.jsonl
    else
      echo "Skipping irrelevant release: $package_name"
    fi
  done
  
  if [[ -f relevant_releases.jsonl ]]; then
    FILTERED=$(cat relevant_releases.jsonl | jq -s)
    echo "Filtered releases: $FILTERED"
    echo "releases=$FILTERED" >> $GITHUB_OUTPUT
  else
    echo "No relevant releases found."
    echo "releases=[]" >> $GITHUB_OUTPUT
  fi

which results in following

Parsing releases from release-plz...
Raw releases data: [{package_name:webterm-core,prs:[],tag:webterm-core-v0.2.1,version:0.2.1},{package_name:webterm-agent,prs:[],tag:webterm-agent-v0.2.1,version:0.2.1},{package_name:webterm-relay,prs:[],tag:webterm-relay-v0.2.1,version:0.2.1}]
jq: parse error: Invalid numeric literal at line 1, column 15
No relevant releases found.

@nasa42 nasa42 changed the title Fix issue with workflow not correctly parsing JSON string. build: Fix issue with workflow not correctly parsing JSON string. Feb 2, 2025
@nasa42 nasa42 merged commit 77a470e into main Feb 2, 2025
6 of 7 checks passed
@nasa42 nasa42 deleted the vik-release-parsing branch February 2, 2025 18:01
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.

1 participant