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

Fix new release GH workflow #1860

Merged
merged 37 commits into from
Jan 11, 2024
Merged

Fix new release GH workflow #1860

merged 37 commits into from
Jan 11, 2024

Conversation

a-zakir
Copy link
Contributor

@a-zakir a-zakir commented Jan 2, 2024

No description provided.

Watermelon AI Summary

AI Summary deactivated by a-zakir

GitHub PRs

Antares_Simulator is an open repo and Watermelon will serve it for free.
🍉🫶

@pull-request-size pull-request-size bot added size/L and removed size/M labels Jan 2, 2024
Copy link

sonarqubecloud bot commented Jan 8, 2024

Please retry analysis of this Pull-Request directly on SonarCloud

Copy link
Member

@flomnes flomnes left a comment

Choose a reason for hiding this comment

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

Can you please add OL8 ?

Comment on lines 21 to 29
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
tag: ${{ github.event.inputs.release_tag }}
title: ${{ github.event.inputs.release_name }}
run: |
gh release create "$tag" \
--repo="$GITHUB_REPOSITORY" \
--title="$title" \
--generate-notes
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
tag: ${{ github.event.inputs.release_tag }}
title: ${{ github.event.inputs.release_name }}
run: |
gh release create "$tag" \
--repo="$GITHUB_REPOSITORY" \
--title="$title" \
--generate-notes
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
tag: ${{ github.event.inputs.release_tag }}
title: ${{ github.event.inputs.release_name }}
tests: ${{ github.event.inputs.run-tests }}
run: |
gh release create "$tag" \
--repo="$GITHUB_REPOSITORY" \
--title="$title" \
--notes="Run tests: $tests"

The generated notes are garbage since our history is not linear. Also, we want to sort changes by type (new feature, bugfix, doc, code cleaning, etc.)

@flomnes
Copy link
Member

flomnes commented Jan 10, 2024

Almost good ! PDF doc is missing

@flomnes flomnes changed the title fix new release Fix new release GH workflow Jan 11, 2024
@a-zakir a-zakir marked this pull request as ready for review January 11, 2024 09:43
gh release create "$tag" \
--repo="$GITHUB_REPOSITORY" \
--title="$title"
--notes="Run tests: $tests"
Copy link
Member

Choose a reason for hiding this comment

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

You forgot to define $tests above (section env)

Copy link
Member

@flomnes flomnes left a comment

Choose a reason for hiding this comment

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

Env variable not defined

@a-zakir a-zakir merged commit c66b890 into develop Jan 11, 2024
9 of 10 checks passed
@a-zakir a-zakir deleted the fix/new_release branch January 11, 2024 09:56
Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

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

Successfully merging this pull request may close these issues.

2 participants